[
https://issues.apache.org/jira/browse/THRIFT-2214?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13814769#comment-13814769
]
Hudson commented on THRIFT-2214:
--------------------------------
SUCCESS: Integrated in Thrift #937 (See
[https://builds.apache.org/job/Thrift/937/])
THRIFT-2214: System header sys/param.h is included inside the Thrift namespace
(bencraig: rev f4e6e62ea091b94322ecc99756269dbee1c06380)
* lib/cpp/src/thrift/protocol/TProtocol.h
> 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)
