[
https://issues.apache.org/jira/browse/THRIFT-3936?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15530363#comment-15530363
]
ASF GitHub Bot commented on THRIFT-3936:
----------------------------------------
Github user nsuke commented on a diff in the pull request:
https://github.com/apache/thrift/pull/1099#discussion_r80976940
--- Diff: lib/cpp/src/thrift/TApplicationException.h ---
@@ -21,6 +21,7 @@
#define _THRIFT_TAPPLICATIONEXCEPTION_H_ 1
#include <thrift/Thrift.h>
+#include <stdint.h>
--- End diff --
As I said, for windows it's OK, for non-windows without stdint.h, it will
fail.
We could improve config.hin and build/cmake/config.h.in to do the same
thing as Windows when boost is avaiable.
> Cannot compile 0.10.0 development tip with VS2013 and earlier (snprintf,
> uint32_t)
> ----------------------------------------------------------------------------------
>
> Key: THRIFT-3936
> URL: https://issues.apache.org/jira/browse/THRIFT-3936
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Library
> Affects Versions: 0.10.0
> Environment: Visual Studio 2013, VS2013, Visual Studio 2010, VS2010,
> Windows
> Reporter: James E. King, III
> Assignee: James E. King, III
> Original Estimate: 0.25h
> Time Spent: 1h
> Remaining Estimate: 0h
>
> A change introduced in July changed some code that won't compile on VS2013:
> https://github.com/apache/thrift/commit/e5176241c325837967a07d6d20e0e9a789fbd107
> snprintf is not available.
> A pull request was submitted for this by someone in the community (thank
> you!):
> https://github.com/apache/thrift/pull/1096
> I recommended that the fix be done using PlatformSocket.h for abstraction of
> the platform; I offered to submit a PR to do this.
> In addition I found that with VS2010 I could not compile anything using
> TApplicationException.h because uint32_t is used within it, but it doesn't
> include stdlib.h.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
