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)