You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Vladimir Strisovsky (JIRA)" <ji...@apache.org> on 2013/09/30 11:18:25 UTC

[jira] [Updated] (THRIFT-2214) System header sys/param.h is included inside the Thrift namespace

     [ https://issues.apache.org/jira/browse/THRIFT-2214?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Vladimir Strisovsky updated THRIFT-2214:
----------------------------------------

    Attachment: 03-header-files.patch

Adding patch for the issue. Please review it and push to main branch. Thank you.

> System header sys/param.h is included inside the Thrift namespace
> -----------------------------------------------------------------
>
>                 Key: THRIFT-2214
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2214
>             Project: Thrift
>          Issue Type: Bug
>          Components: C++ - Library
>    Affects Versions: 0.9.1
>            Reporter: Vladimir Strisovsky
>         Attachments: 03-header-files.patch
>
>
> #include <sys/param.h> is in apache::thrift::protocol namespace. It causes compilation error when param.h is already included or header is included later.
> see:
> namespace apache { namespace thrift { namespace protocol {
> using apache::thrift::transport::TTransport;
> #ifdef HAVE_SYS_PARAM_H
> #include <sys/param.h>
> #endif



--
This message was sent by Atlassian JIRA
(v6.1#6144)