[
https://issues.apache.org/jira/browse/THRIFT-2045?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13790521#comment-13790521
]
Hudson commented on THRIFT-2045:
--------------------------------
FAILURE: Integrated in Thrift #925 (See
[https://builds.apache.org/job/Thrift/925/])
Revert "THRIFT-2045: C++ compiler warning and build cleanup" (bencraig: rev
9f9cd10e813ef574dd5578d78ca26a9088383d3a)
* compiler/cpp/src/generate/t_cpp_generator.cc
* compiler/cpp/src/thriftl.ll
* compiler/cpp/src/windows/version.h.in
* compiler/cpp/src/windows/config.h
* compiler/cpp/src/platform.h
* compiler/cpp/src/main.cc
* compiler/cpp/src/generate/t_go_generator.cc
* compiler/cpp/src/thrifty.yy
* compiler/cpp/src/parse/t_field.h
* compiler/cpp/src/generate/t_py_generator.cc
> C++ compiler warning and build cleanup
> --------------------------------------
>
> Key: THRIFT-2045
> URL: https://issues.apache.org/jira/browse/THRIFT-2045
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Compiler, Go - Compiler, Python - Compiler
> Affects Versions: 0.9
> Reporter: Ben Craig
> Assignee: Ben Craig
> Priority: Minor
> Fix For: 1.0
>
> Attachments: windows_compiler_port.patch
>
>
> Fixing assorted warnings and minor issues in the compiler:
> * Fixing a warning in C++ generated code concerning an unused catch argument
> in oneway functions.
> * Generally using _WIN32 instead of MINGW as a test for Windows. Building
> the compiler should no longer require the build process to define MINGW.
> * Providing THRIFT_PATH_MAX for the portable portions of the code. This
> evaluates to either MAX_PATH or PATH_MAX, depending on whichever one is
> available.
> * Shrank "value" field of t_field_id in order to clean up warnings. I
> suspect that thrift doesn't support more than 2 billion fields anyway.
> * Squelching various warnings in code, instead of in the build process. FOr
> example, "unreferenced label", "switch statement contains 'default' but no
> 'case' labels", and "bool conversion performance warnings" are all disabled
> via pragma now, at appropriate locations.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
