[
https://issues.apache.org/jira/browse/THRIFT-2907?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14254244#comment-14254244
]
ASF GitHub Bot commented on THRIFT-2907:
----------------------------------------
GitHub user jparise opened a pull request:
https://github.com/apache/thrift/pull/334
Don't define ntohll() if it is already defined.
Fixes [THRIFT-2907](https://issues.apache.org/jira/browse/THRIFT-2907)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jparise/thrift thrift-2907
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/334.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #334
----
commit 68cc8358b14d184bb5faac4faca7ea9d5b0ce308
Author: Jon Parise <[email protected]>
Date: 2014-12-19T23:32:24Z
Don't define ntohll() if it is already defined.
Fixes THRIFT-2907
----
> 'ntohll' macro redefined
> ------------------------
>
> Key: THRIFT-2907
> URL: https://issues.apache.org/jira/browse/THRIFT-2907
> Project: Thrift
> Issue Type: Bug
> Components: Build Process
> Affects Versions: 0.9.2
> Environment: Apple LLVM version 6.0 (clang-600.0.54) (based on LLVM
> 3.5svn)
> Target: x86_64-apple-darwin14.0.0
> Thread model: posix
> Reporter: Jon Parise
> Priority: Trivial
>
> The {{ntohll}} macro is already defined in {{/usr/include/sys/_endian.h}} on
> Mac OS X.
> {code}
> In file included from gen-cpp/shared_constants.cpp:7:
> In file included from gen-cpp/shared_constants.h:10:
> In file included from gen-cpp/shared_types.h:14:
> ../../lib/cpp/src/thrift/protocol/TProtocol.h:131:11: warning: 'ntohll' macro
> redefined
> # define ntohll(n) ( (((uint64_t)ntohl((uint32_t)n)) << 32) +
> ntohl((uint32_t)(n >> 32)) )
> ^
> /usr/include/sys/_endian.h:140:9: note: previous definition is here
> #define ntohll(x) __DARWIN_OSSwapInt64(x)
> ^
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
