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

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.9.1
• Fix Version/s: 0.9.2
• Component/s: C++ - Library
• Assignee: Ben Craig
• Reporter: Vladimir Strisovsky
• Created: 30/Sep/13 09:10
• Updated: 06/Nov/13 13:19
• Resolved: 06/Nov/13 01:51

Description

#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