You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Ben Craig (JIRA)" <ji...@apache.org> on 2013/11/06 02:52:20 UTC

[jira] [Resolved] (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 ]

Ben Craig resolved THRIFT-2214.
-------------------------------

       Resolution: Fixed
    Fix Version/s: 0.9.2

Committed.
Thanks Vladmir!

> 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
>            Assignee: Ben Craig
>             Fix For: 0.9.2
>
>         Attachments: 03-header-files.patch, 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)