[
https://issues.apache.org/jira/browse/THRIFT-916?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12933146#action_12933146
]
Piotr Bartosiewicz commented on THRIFT-916:
-------------------------------------------
I applied both patches and got some errors:
{noformat}
src/protocol/TBinaryProtocol.tcc:390:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
src/protocol/TBinaryProtocol.tcc:359:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
src/protocol/TBinaryProtocol.tcc:368:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
src/protocol/TBinaryProtocol.tcc:377:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
src/transport/TFileTransport.cpp:673:56: error: dereferencing type-punned
pointer will break strict-aliasing rules
src/transport/TFileTransport.cpp:684:84: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:359:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:368:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:377:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:390:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:359:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:368:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:377:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:390:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:359:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:368:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:377:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
./src/protocol/TBinaryProtocol.tcc:390:3: error: dereferencing type-punned
pointer will break strict-aliasing rules
../../../lib/cpp/src/protocol/TBinaryProtocol.tcc:359:3: error: dereferencing
type-punned pointer will break strict-aliasing rules
../../../lib/cpp/src/protocol/TBinaryProtocol.tcc:368:3: error: dereferencing
type-punned pointer will break strict-aliasing rules
../../../lib/cpp/src/protocol/TBinaryProtocol.tcc:377:3: error: dereferencing
type-punned pointer will break strict-aliasing rules
../../../lib/cpp/src/protocol/TBinaryProtocol.tcc:390:3: error: dereferencing
type-punned pointer will break strict-aliasing rules
{noformat}
{noformat}
x86_64 GNU/Linux
g++ (Ubuntu/Linaro 4.5.1-7ubuntu2) 4.5.1
flex 2.5.35
{noformat}
> gcc warnings in c++ header files
> --------------------------------
>
> Key: THRIFT-916
> URL: https://issues.apache.org/jira/browse/THRIFT-916
> Project: Thrift
> Issue Type: Improvement
> Components: C++ - Library
> Affects Versions: 0.4
> Reporter: Piotr Bartosiewicz
> Assignee: Roger Meier
> Attachments: DebianLenny_error.log, make-k.log,
> THRIFT-916_Wall_pedantic__noerrors.patch,
> THRIFT-916_Wall_Wextra__NOWARNINGS_DebianLenny.patch,
> v1-enable-wall-werror.patch, v1-fix-format-strings.patch,
> v1-fix-java-style-and-warnings.patch, v1-fix-more-warnings.patch,
> v1-fix-random-silly-warnings.patch, v2-fix-all-warnings.patch,
> v3-fix-all-warnings.patch, v4-fix-all-warnings.patch
>
>
> I tried to add an extra gcc warning options in my project but a number of
> warnings was emmited from thrift headers.
> Following the boost library recommendations, code should be clean with "-Wall
> -Wextra -pedantic" options.
> Currently only with -Wall thrift has no warnings.
> Other options worth considering (but require more work) are -Wconversion
> -Wold-style-cast
> So the task is to improve the thrift headers.
> I suggest also update all the thrift makefiles so the thrift sources an tests
> will indicate the faulty code.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
