Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-1192

Typo: TProtocol.h tests for HAVE_SYS_PARAM_H_

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.6.1, 0.7
    • 0.7
    • None
    • None
    • Patch Available

    Description

      TProtocol.h tests for HAVE_SYS_PARAM_H_, which will never be defined, so <sys/param.h> is never included.

      Attached is a patch that fixes the typo on HAVE_SYS_PARAM_H, but I actually suspect that <sys/param.h> is not needed at all in the current codebase, as nobody reported a problem and I can't find any obvious need for it.

      Attachments

        1. thrift-have-sys-param-typo.patch
          0.4 kB
          Eric Rannaud

        Activity

          People

            eric.rannaud Eric Rannaud
            eric.rannaud Eric Rannaud
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: