Repository: thrift
Updated Branches:
  refs/heads/master 9b0c47f77 -> cb0754db1


THRIFT-2784 Eliminate compiler warnings in generated C++ code

remove unused MyEnum5 and add -pedantic to lib/cpp/test/Makefile.am


Project: http://git-wip-us.apache.org/repos/asf/thrift/repo
Commit: http://git-wip-us.apache.org/repos/asf/thrift/commit/cb0754db
Tree: http://git-wip-us.apache.org/repos/asf/thrift/tree/cb0754db
Diff: http://git-wip-us.apache.org/repos/asf/thrift/diff/cb0754db

Branch: refs/heads/master
Commit: cb0754db1e73e7ea347aae07e0ef1de9792ef99e
Parents: 9b0c47f
Author: Roger Meier <[email protected]>
Authored: Tue Oct 21 21:28:04 2014 +0200
Committer: Roger Meier <[email protected]>
Committed: Tue Oct 21 21:28:04 2014 +0200

----------------------------------------------------------------------
 lib/cpp/test/Makefile.am | 2 +-
 test/EnumTest.thrift     | 6 ------
 2 files changed, 1 insertion(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/thrift/blob/cb0754db/lib/cpp/test/Makefile.am
----------------------------------------------------------------------
diff --git a/lib/cpp/test/Makefile.am b/lib/cpp/test/Makefile.am
index 751b0b3..f15b0c1 100755
--- a/lib/cpp/test/Makefile.am
+++ b/lib/cpp/test/Makefile.am
@@ -268,7 +268,7 @@ gen-cpp/ChildService.cpp: processor/proc.thrift
 
 AM_CPPFLAGS = $(BOOST_CPPFLAGS) -I$(top_srcdir)/lib/cpp/src
 AM_LDFLAGS = $(BOOST_LDFLAGS)
-AM_CXXFLAGS = -Wall -Wextra
+AM_CXXFLAGS = -Wall -Wextra -pedantic
 
 clean-local:
        $(RM) -r gen-cpp

http://git-wip-us.apache.org/repos/asf/thrift/blob/cb0754db/test/EnumTest.thrift
----------------------------------------------------------------------
diff --git a/test/EnumTest.thrift b/test/EnumTest.thrift
index deb852f..6201923 100644
--- a/test/EnumTest.thrift
+++ b/test/EnumTest.thrift
@@ -64,12 +64,6 @@ enum MyEnum4 {
   // represented with an i32.
 }
 
-enum MyEnum5 {
-  // attempting to explicitly use values out of the i32 range will also fail
-  // ME5_A = 0x80000000,
-  // ME5_B = 0x100000000,
-}
-
 struct MyStruct {
   1: MyEnum2 me2_2 = MyEnum1.ME2_2
   2: MyEnum3 me3_n2 = MyEnum3.ME3_N2

Reply via email to