[ https://issues.apache.org/jira/browse/PROTON-546?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15410005#comment-15410005 ]
Chuck Rolke commented on PROTON-546: ------------------------------------ You can see the warnings that gcc would throw if they weren't suppressed. {noformat} diff --git a/proton-c/CMakeLists.txt b/proton-c/CMakeLists.txt index 0fec525..46b3a0a 100644 --- a/proton-c/CMakeLists.txt +++ b/proton-c/CMakeLists.txt @@ -209,7 +209,7 @@ if (CMAKE_COMPILER_IS_GNUCC) if (ENABLE_WARNING_ERROR) set (WERROR "-Werror") endif (ENABLE_WARNING_ERROR) - set (COMPILE_WARNING_FLAGS "${WERROR} -Wall -pedantic-errors") + set (COMPILE_WARNING_FLAGS "-Wconversion -Wall -pedantic-errors") # C++ allow "%z" format specifier and variadic macros set (CXX_WARNING_FLAGS "${COMPILE_WARNING_FLAGS} -Wno-format -Wno-variadic-macros") if (NOT BUILD_WITH_CXX) {noformat} This turns off -Werror so that all files compile. Most of the warnings are totally harmless and reflect the nature of regular C. There are a few silent demotions/promotions of long and unsigned long to int that may or may not be an issue. In a recent compile with GCC 5.3.1 x86_64 vs Visual Studio 2015 64-bit there was one int size difference of consequence that proton code does not code around: in gcc a sizeof(long)=8 while in windows sizeof(long)=4. Same goes for the unsigned long. ||Test||gcc||Visual Studio|| |sizeof(int )| 4 | 4 | |sizeof(unsigned int )| 4 | 4 | |sizeof(char )| 1 | 1 | |sizeof(unsigned char )| 1 | 1 | |sizeof(short )| 2 | 2 | |sizeof(unsigned short )| 2 | 2 | |sizeof(long )| 8 | 4 | |sizeof(unsigned long )| 8 | 4 | |sizeof(long long )| 8 | 8 | |sizeof(unsigned long long)| 8 | 8 | |sizeof(int8_t )| 1 | 1 | |sizeof(uint8_t )| 1 | 1 | |sizeof(int16_t )| 2 | 2 | |sizeof(uint16_t )| 2 | 2 | |sizeof(int32_t )| 4 | 4 | |sizeof(uint32_t )| 4 | 4 | |sizeof(int64_t )| 8 | 8 | |sizeof(uint64_t )| 8 | 8 | > C++ Windows warnings: conversions and possible loss of data > ----------------------------------------------------------- > > Key: PROTON-546 > URL: https://issues.apache.org/jira/browse/PROTON-546 > Project: Qpid Proton > Issue Type: Bug > Components: proton-c > Affects Versions: 0.6 > Reporter: Chuck Rolke > Assignee: Andrew Stitcher > Labels: windows > Attachments: PROTON-546-size-warnings.txt > > > Windows has other warnings similar to PROTON-545, especially during 64-bit > builds. > See attached file for details. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org