On 05/19/2017 02:44 PM, Gregory Szorc wrote:
`mach build` attempts to parse compiler warnings to a persisted "database."
You can view a list of compiler warnings post build by running `mach
warnings-list`. The intent behind this feature was to make it easier to
find and fix compiler warnings. After all, something out of sight is out of
mind.

There have been a few recent changes to increase the visibility of compiler
warnings with the expectation being that raising visibility will increase
the chances of someone addressing them. After all, a compiler warning is
either a) valid and should be fixed or b) invalid and should be ignored.
Either way, a compiler warning shouldn't exist.

Since mystor and billm's objection seems to have gone unaddressed in this thread, let me try once again. There is a (c) scenario that you are ignoring here:

c) the warning is valid and should be fixed but it's not more important than other things than the developer may want to do at the moment. I would like to posit that this is often the case. Let's look at what I get these days at the end of a normal debug build on Linux. This output is long and I am intentionally pasting the full thing here in order to make a point on how much useless information we are presenting at the end of each build:

18:39.34 warning: db/sqlite3/src/sqlite3.c:131236:39 [-Wunreachable-code] code will never be executed 18:39.34 warning: db/sqlite3/src/sqlite3.c:131292:39 [-Wunreachable-code] code will never be executed 18:39.34 warning: db/sqlite3/src/sqlite3.c:137670:9 [-Wunreachable-code] code will never be executed 18:39.34 warning: gfx/angle/src/compiler/translator/ASTMetadataHLSL.cpp:93:15 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.34 warning: gfx/angle/src/compiler/translator/ParseContext.cpp:1123:9 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.34 warning: gfx/angle/src/compiler/translator/ParseContext.cpp:3640:9 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.34 warning: gfx/angle/src/compiler/translator/ParseContext.cpp:3808:9 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.34 warning: gfx/cairo/libpixman/src/pixman-bits-image.c:268:32 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.34 warning: gfx/cairo/libpixman/src/pixman-linear-gradient.c:395:6 [-Wunreachable-code] code will never be executed 18:39.34 warning: gfx/cairo/libpixman/src/pixman-x86.c:80:5 [-Wexpansion-to-defined] macro expansion producing 'defined' has undefined behavior 18:39.34 warning: gfx/cairo/libpixman/src/pixman-x86.c:119:5 [-Wexpansion-to-defined] macro expansion producing 'defined' has undefined behavior 18:39.34 warning: intl/hyphenation/hyphen/hyphen.c:332:27 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.34 warning: intl/icu/source/common/locdspnm.cpp:286:14 [-Wunused-private-field] private field 'capitalizationBrkIter' is not used 18:39.34 warning: intl/icu/source/common/udataswp.c:438:29 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/common/ulist.c:161:24 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/common/uloc_tag.c:1374:31 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/common/uloc_tag.c:1409:36 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/common/ures_cnv.c:46:18 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/common/ures_cnv.c:64:22 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/common/ushape.cpp:1247:12 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustring.cpp:860:57 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustring.cpp:870:57 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustrtrns.cpp:488:47 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustrtrns.cpp:535:47 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustrtrns.cpp:609:51 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustrtrns.cpp:655:47 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/ustrtrns.cpp:704:47 [-Wcomma] possible misuse of comma operator here 18:39.34 warning: intl/icu/source/common/utrace.c:149:16 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: intl/icu/source/i18n/decNumber.c:5010:21 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'unsigned long' 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:224:16 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:245:20 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:264:20 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:270:16 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:323:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:494:18 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: js/src/ctypes/libffi/src/x86/ffi64.c:669:18 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.34 warning: media/ffvpx/libavcodec/utils.c:2788:15 [-Wdeprecated-declarations] 'avcodec_decode_video2' is deprecated 18:39.34 warning: media/ffvpx/libavcodec/utils.c:2793:15 [-Wdeprecated-declarations] 'avcodec_decode_audio4' is deprecated 18:39.34 warning: media/ffvpx/libavcodec/utils.c:2939:15 [-Wdeprecated-declarations] 'avcodec_encode_video2' is deprecated 18:39.35 warning: media/ffvpx/libavcodec/utils.c:2942:15 [-Wdeprecated-declarations] 'avcodec_encode_audio2' is deprecated 18:39.35 warning: media/ffvpx/libavutil/dummy_funcs.c:83:34 [-Wvisibility] declaration of 'struct AVBufferRef' will not be visible outside of this function 18:39.35 warning: media/ffvpx/libavutil/dummy_funcs.c:83:67 [-Wvisibility] declaration of 'struct AVFrame' will not be visible outside of this function 18:39.35 warning: media/ffvpx/libavutil/utils.c:119:26 [-Wstring-conversion] implicit conversion turns string literal into bool: 'char [19]' to '_Bool' 18:39.35 warning: media/libmkv/EbmlBufferWriter.c:43:13 [-Wunused-function] unused function '_Serialize' 18:39.35 warning: media/libsoundtouch/src/InterpolateShannon.cpp:75:9 [-Wmacro-redefined] 'PI' macro redefined 18:39.35 warning: media/libsoundtouch/src/cpu_detect_x86.cpp:55:12 [-Wmacro-redefined] 'bit_MMX' macro redefined 18:39.35 warning: media/libsoundtouch/src/cpu_detect_x86.cpp:56:12 [-Wmacro-redefined] 'bit_SSE' macro redefined 18:39.35 warning: media/libsoundtouch/src/cpu_detect_x86.cpp:57:12 [-Wmacro-redefined] 'bit_SSE2' macro redefined 18:39.35 warning: media/libspeex_resampler/src/resample.c:814:17 [-Wunused-variable] unused variable 'i' 18:39.35 warning: media/libtheora/lib/decode.c:195:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:198:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:208:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:210:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:212:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:214:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:216:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:220:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:221:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:227:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:228:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:229:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:230:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:236:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:237:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:238:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:239:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:242:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:253:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:257:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:260:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:262:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:264:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:266:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:270:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:271:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:277:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:278:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:279:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/decode.c:280:3 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.35 warning: media/libtheora/lib/x86/x86state.c:24:28 [-Wunused-const-variable] unused variable 'OC_FZIG_ZAG_MMX' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_component.c:998:5 [-Wmissing-prototypes] no previous prototype for function 'nr_ice_component_can_candidate_tcptype_pair' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_component.c:1015:5 [-Wmissing-prototypes] no previous prototype for function 'nr_ice_component_can_candidate_addr_pair' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_component.c:1152:5 [-Wmissing-prototypes] no previous prototype for function 'nr_ice_pre_answer_enqueue' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_component.c:1326:5 [-Wmissing-prototypes] no previous prototype for function 'nr_ice_component_refresh_consent' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_component.c:1340:6 [-Wmissing-prototypes] no previous prototype for function 'nr_ice_component_consent_calc_consent_timer' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_component.c:1433:5 [-Wmissing-prototypes] no previous prototype for function 'nr_ice_component_setup_consent' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_ctx.c:919:14 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_media_stream.c:567:18 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_media_stream.c:568:27 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/ice/ice_media_stream.c:730:5 [-Wimplicit-function-declaration] implicit declaration of function 'nr_ice_component_consent_destroy' is invalid in C99 18:39.35 warning: media/mtransport/third_party/nICEr/src/net/transport_addr.c:478:26 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_build.c:328:53 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_client_ctx.c:246:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'UINT4' (aka 'unsigned int') 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_client_ctx.c:256:5 [-Wimplicit-function-declaration] implicit declaration of function 'nr_ice_accumulate_count' is invalid in C99 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_client_ctx.c:282:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'UINT4' (aka 'unsigned int') 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_client_ctx.c:405:29 [-Wsign-compare] comparison of integers of different signs: 'int' and 'UINT4' (aka 'unsigned int') 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:92:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:108:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:124:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:155:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:172:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:189:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:231:25 [-Wsign-compare] comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:564:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:640:21 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:971:24 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:974:28 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:1220:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:1393:40 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_codec.c:1467:39 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_hint.c:70:39 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_hint.c:170:39 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_hint.c:187:39 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_hint.c:204:39 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_hint.c:224:39 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:77:16 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:212:9 [-Wunused-value] ignoring return value of function declared with pure attribute 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:228:56 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:240:7 [-Wunused-value] ignoring return value of function declared with pure attribute 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:247:7 [-Wunused-value] ignoring return value of function declared with pure attribute 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:254:7 [-Wunused-value] ignoring return value of function declared with pure attribute 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_msg.c:268:7 [-Wunused-value] ignoring return value of function declared with pure attribute 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_util.c:97:29 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/stun_util.c:217:24 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/turn_client_ctx.c:260:9 [-Wimplicit-function-declaration] implicit declaration of function 'nr_ice_accumulate_count' is invalid in C99 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/turn_client_ctx.c:904:18 [-Wsign-compare] comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' 18:39.35 warning: media/mtransport/third_party/nICEr/src/stun/turn_client_ctx.c:905:15 [-Wsign-compare] comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' 18:39.35 warning: media/mtransport/third_party/nrappkit/src/log/r_log.c:199:63 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nrappkit/src/log/r_log.c:206:43 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.35 warning: media/mtransport/third_party/nrappkit/src/log/r_log.c:569:41 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/log/r_log.c:587:41 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/c2ru.c:239:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registry.c:362:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registry.c:579:27 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registry_local.c:885:9 [-Wmacro-redefined] 'NRREGGET' macro redefined 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registry_local.c:978:9 [-Wmacro-redefined] 'NRREGSET' macro redefined 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registrycb.c:106:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registrycb.c:402:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/registry/registrycb.c:425:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/util/hex.c:59:14 [-Wsign-compare] comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/util/hex.c:67:14 [-Wsign-compare] comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/util/hex.c:92:14 [-Wsign-compare] comparison of integers of different signs: 'size_t' (aka 'unsigned long') and 'int' 18:39.36 warning: media/mtransport/third_party/nrappkit/src/util/libekr/r_errors.c:117:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.36 warning: media/webrtc/trunk/third_party/gflags/src/gflags.cc:281:53 [-Wunused-local-typedef] unused typedef 'true_false_equal' 18:39.36 warning: media/webrtc/trunk/third_party/gflags/src/gflags_completions.cc:327:5 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.36 warning: media/webrtc/trunk/third_party/gflags/src/gflags_completions.cc:328:5 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.36 warning: media/webrtc/trunk/third_party/gflags/src/gflags_reporting.cc:137:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.36 warning: media/webrtc/trunk/webrtc/call/call.cc:97:16 [-Winconsistent-missing-override] 'voice_engine' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/call/congestion_controller.cc:92:8 [-Winconsistent-missing-override] 'SetMinBitrate' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/common_audio/resampler/resampler.cc:54:28 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.36 warning: media/webrtc/trunk/webrtc/modules/audio_coding/neteq/background_noise.h:98:22 [-Wcomma] possible misuse of comma operator here 18:39.36 warning: media/webrtc/trunk/webrtc/modules/audio_coding/neteq/expand.cc:702:43 [-Wshift-negative-value] shifting a negative signed value is undefined 18:39.36 warning: media/webrtc/trunk/webrtc/modules/audio_processing/aecm/aecm_defines.h:23:9 [-Wmacro-redefined] 'PART_LEN2' macro redefined 18:39.36 warning: media/webrtc/trunk/webrtc/modules/desktop_capture/desktop_device_info.cc:188:27 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'size_type' (aka 'unsigned long') 18:39.36 warning: media/webrtc/trunk/webrtc/modules/desktop_capture/desktop_device_info.cc:207:28 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'size_type' (aka 'unsigned long') 18:39.36 warning: media/webrtc/trunk/webrtc/modules/desktop_capture/desktop_device_info.cc:228:27 [-Wsign-compare] comparison of integers of different signs: 'int32_t' (aka 'int') and 'size_type' (aka 'unsigned long') 18:39.36 warning: media/webrtc/trunk/webrtc/modules/desktop_capture/differ_unittest.cc:187:22 [-Wcomma] possible misuse of comma operator here 18:39.36 warning: media/webrtc/trunk/webrtc/modules/desktop_capture/x11/desktop_device_info_x11.cc:85:21 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and '__pid_t' (aka 'int') 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/h264_sps_parser.cc:17:9 [-Wmacro-redefined] 'RETURN_FALSE_ON_FAIL' macro redefined 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtcp_packet/tmmbn.h:47:10 [-Winconsistent-missing-override] 'BlockLength' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtcp_sender.cc:517:19 [-Wrange-loop-analysis] loop variable 'it' of type 'const std::pair<const unsigned int, std::__cxx11::basic_string<char> >' creates a copy from type 'const std::pair<const unsigned int, std::__cxx11::basic_string<char> >' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc:47:6 [-Wunused-function] unused function 'VerifyStapANaluLengths' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc:100:5 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc:109:5 [-Wimplicit-fallthrough] unannotated fall-through between switch labels 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_payload_registry.cc:423:7 [-Winconsistent-missing-override] 'GetPayloadTypeFrequency' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_payload_registry.cc:476:7 [-Winconsistent-missing-override] 'GetPayloadTypeFrequency' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h:37:8 [-Winconsistent-missing-override] 'SetTelephoneEventForwardToDecoder' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h:40:8 [-Winconsistent-missing-override] 'TelephoneEventForwardToDecoder' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h:43:8 [-Winconsistent-missing-override] 'TelephoneEventPayloadType' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_receiver_audio.h:45:26 [-Winconsistent-missing-override] 'GetTelephoneEventHandler' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/rtp_rtcp/source/rtp_receiver_video.h:37:26 [-Winconsistent-missing-override] 'GetTelephoneEventHandler' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:48:21 [-Winconsistent-missing-override] 'RegisterModuleFileCallback' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:49:25 [-Winconsistent-missing-override] 'RecordingFileFormat' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:58:21 [-Winconsistent-missing-override] 'StopRecording' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:59:18 [-Winconsistent-missing-override] 'IsRecording' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:60:21 [-Winconsistent-missing-override] 'codec_info' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:61:21 [-Winconsistent-missing-override] 'RecordAudioToFile' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/utility/source/file_recorder_impl.h:72:21 [-Winconsistent-missing-override] 'RecordVideoToFile' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/realtime_temporal_layers.cc:104:16 [-Winconsistent-missing-override] 'ConfigureBitrates' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/realtime_temporal_layers.cc:187:15 [-Winconsistent-missing-override] 'EncodeFlags' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/realtime_temporal_layers.cc:199:15 [-Winconsistent-missing-override] 'CurrentLayerId' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/realtime_temporal_layers.cc:206:16 [-Winconsistent-missing-override] 'PopulateCodecSpecific' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/vp8_impl.h:43:15 [-Winconsistent-missing-override] 'Release' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/vp8_impl.h:45:15 [-Winconsistent-missing-override] 'InitEncode' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/vp8_impl.h:49:15 [-Winconsistent-missing-override] 'Encode' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/vp8_impl.h:53:15 [-Winconsistent-missing-override] 'RegisterEncodeCompleteCallback' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/vp8_impl.h:55:15 [-Winconsistent-missing-override] 'SetChannelParameters' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/codecs/vp8/vp8_impl.h:57:15 [-Winconsistent-missing-override] 'SetRates' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/modules/video_coding/generic_decoder.h:106:8 [-Wunused-private-field] private field '_keyFrameDecoded' is not used 18:39.36 warning: media/webrtc/trunk/webrtc/video/vie_channel.h:243:19 [-Winconsistent-missing-override] 'FrameToRender' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/video/vie_channel.h:246:19 [-Winconsistent-missing-override] 'ReceivedDecodedReferenceFrame' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/video/vie_channel.h:259:16 [-Winconsistent-missing-override] 'OnDecoderTiming' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/video/vie_channel.h:268:19 [-Winconsistent-missing-override] 'RequestKeyFrame' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/video/vie_channel.h:271:19 [-Winconsistent-missing-override] 'SliceLossIndicationRequest' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/video_engine/desktop_capture_impl.cc:506:3 [-Wreorder] field 'mRefCount' will be initialized after field 'capturer_thread_' 18:39.36 warning: media/webrtc/trunk/webrtc/voice_engine/channel.cc:1200:39 [-Wcomma] possible misuse of comma operator here 18:39.36 warning: media/webrtc/trunk/webrtc/voice_engine/voe_external_media_impl.h:29:15 [-Winconsistent-missing-override] 'SetExternalRecordingStatus' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/voice_engine/voe_external_media_impl.h:31:15 [-Winconsistent-missing-override] 'SetExternalPlayoutStatus' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/voice_engine/voe_external_media_impl.h:33:15 [-Winconsistent-missing-override] 'ExternalRecordingInsertData' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/voice_engine/voe_external_media_impl.h:40:15 [-Winconsistent-missing-override] 'ExternalPlayoutData' overrides a member function but is not marked 'override' 18:39.36 warning: media/webrtc/trunk/webrtc/voice_engine/voe_external_media_impl.h:47:15 [-Winconsistent-missing-override] 'ExternalPlayoutGetData' overrides a member function but is not marked 'override' 18:39.36 warning: modules/woff2/src/store_bytes.h:60:27 [-Wunused-value] expression result unused 18:39.36 warning: modules/woff2/src/woff2_dec.cc:1149:12 [-Wunused-variable] unused variable 'dst_offset' 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:240:6 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:389:6 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:520:6 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:1303:13 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:1512:13 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:1918:2 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:2856:14 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:3415:6 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:3467:5 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_asconf.c:3516:4 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_input.c:1670:15 [-Waddress-of-packed-member] taking address of packed member 'time_entered' of class or structure 'sctp_state_cookie' may result in an unaligned pointer value 18:39.36 warning: netwerk/sctp/src/netinet/sctp_input.c:2412:10 [-Waddress-of-packed-member] taking address of packed member 'time_entered' of class or structure 'sctp_state_cookie' may result in an unaligned pointer value 18:39.36 warning: netwerk/sctp/src/netinet/sctp_output.c:5921:30 [-Waddress-of-packed-member] taking address of packed member 'time_entered' of class or structure 'sctp_state_cookie' may result in an unaligned pointer value 18:39.36 warning: netwerk/sctp/src/netinet/sctp_pcb.c:2765:6 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_pcb.c:4154:3 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_pcb.c:4156:3 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_pcb.c:4158:3 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_usrreq.c:6659:3 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/netinet/sctp_usrreq.c:6710:3 [-Wunreachable-code] code will never be executed 18:39.36 warning: netwerk/sctp/src/user_socket.c:2383:27 [-Wunreachable-code-loop-increment] loop will run at most once (loop increment never executed) 18:39.36 warning: nsprpub/lib/libc/src/plgetopt.c:163:46 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'PRIntn' (aka 'int') 18:39.36 warning: nsprpub/pr/src/io/priometh.c:336:24 [-Wsign-compare] comparison of integers of different signs: 'PROffset32' (aka 'int') and 'unsigned long' 18:39.36 warning: nsprpub/pr/src/io/prprf.c:323:31 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/io/prprf.c:926:9 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/md/unix/unix.c:3560:16 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PROffset32' (aka 'int') 18:39.37 warning: nsprpub/pr/src/misc/prdtoa.c:571:50 [-Wsign-compare] comparison of integers of different signs: 'long' and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/misc/prdtoa.c:2627:52 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.37 warning: nsprpub/pr/src/misc/prnetdb.c:677:17 [-Wsign-compare] comparison of integers of different signs: 'PRIntn' (aka 'int') and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/misc/prnetdb.c:855:17 [-Wsign-compare] comparison of integers of different signs: 'PRIntn' (aka 'int') and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/misc/prnetdb.c:1048:17 [-Wsign-compare] comparison of integers of different signs: 'PRIntn' (aka 'int') and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:237:39 [-Wsign-compare] comparison of integers of different signs: 'PRUintn' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:733:12 [-Wsign-compare] comparison of integers of different signs: 'int' and 'PRSize' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:752:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'PRSize' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:776:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'PRSize' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:881:39 [-Wsign-compare] comparison of integers of different signs: 'PRIntn' (aka 'int') and 'PRSize' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:884:23 [-Wsign-compare] comparison of integers of different signs: 'PRIntn' (aka 'int') and 'size_t' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:936:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'PRSize' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:1116:12 [-Wsign-compare] comparison of integers of different signs: 'ssize_t' (aka 'long') and 'size_t' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:1245:32 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'PRInt32' (aka 'int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:1253:27 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'PRInt32' (aka 'int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:1395:27 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'size_t' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:2735:16 [-Wsign-compare] comparison of integers of different signs: 'ssize_t' (aka 'long') and 'size_t' (aka 'unsigned long') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:2777:25 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:3898:22 [-Wsign-compare] comparison of integers of different signs: 'PRIntn' (aka 'int') and 'PRUint32' (aka 'unsigned int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:3965:43 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'PRInt32' (aka 'int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptio.c:4850:24 [-Wsign-compare] comparison of integers of different signs: 'rlim_t' (aka 'unsigned long') and 'PRIntn' (aka 'int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptsynch.c:1001:47 [-Wclass-varargs] passing object of class type 'union semun' through variadic function 18:39.37 warning: nsprpub/pr/src/pthreads/ptsynch.c:1038:41 [-Wclass-varargs] passing object of class type 'union semun' through variadic function 18:39.37 warning: nsprpub/pr/src/pthreads/ptsynch.c:1118:36 [-Wclass-varargs] passing object of class type 'union semun' through variadic function 18:39.37 warning: nsprpub/pr/src/pthreads/ptthread.c:233:29 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'PRUintn' (aka 'unsigned int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptthread.c:497:37 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'PRUintn' (aka 'unsigned int') 18:39.37 warning: nsprpub/pr/src/pthreads/ptthread.c:1109:29 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'PRUintn' (aka 'unsigned int') 18:39.37 warning: obj-ff-clang-plugin.noindex/dist/include/google/protobuf/stubs/map_util.h:356:43 [-Wunused-local-typedef] unused typedef 'value_type' 18:39.37 warning: obj-ff-clang-plugin.noindex/dist/include/google/protobuf/stubs/map_util.h:367:43 [-Wunused-local-typedef] unused typedef 'value_type' 18:39.37 warning: obj-ff-clang-plugin.noindex/dist/include/mozilla/CountingAllocatorBase.h:125:5 [-Wundefined-var-template] instantiation of variable 'mozilla::CountingAllocatorBase<HunspellAllocator>::sAmount' required here, but no definition is available 18:39.37 warning: security/nss/cmd/lib/basicutil.c:636:30 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.37 warning: security/nss/cmd/lib/basicutil.c:640:30 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.37 warning: security/nss/cmd/lib/secutil.c:3444:62 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned int' 18:39.37 warning: security/nss/cmd/shlibsign/shlibsign.c:151:22 [-Wsign-compare] comparison of integers of different signs: 'int' and 'CK_ULONG' (aka 'unsigned long') 18:39.37 warning: security/nss/lib/certhigh/certvfy.c:498:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/ckfw/builtins/bfind.c:117:22 [-Wsign-compare] comparison of integers of different signs: 'int' and 'CK_ULONG' (aka 'unsigned long') 18:39.37 warning: security/nss/lib/ckfw/wrap.c:225:9 [-Wunreachable-code] code will never be executed 18:39.37 warning: security/nss/lib/dev/ckhelper.c:135:45 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/dev/ckhelper.c:445:36 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/freebl/ec.c:30:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/freebl/ec.c:104:26 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/ec.c:472:33 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/ecl/ecp_25519.c:88:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/freebl/gcm.c:227:13 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.37 warning: security/nss/lib/freebl/mpi/mpi.c:2125:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/freebl/mpi/mpi.c:2127:15 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/freebl/mpi/mpi.c:4617:13 [-Wsign-compare] comparison of integers of different signs: 'mp_digit' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/freebl/mpi/mplogic.c:237:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/freebl/mpi/mplogic.c:268:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/freebl/pqg.c:1644:50 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/pqg.c:1702:30 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/pqg.c:1763:29 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/rsa.c:251:50 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/rsa.c:255:43 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.37 warning: security/nss/lib/freebl/shvfy.c:273:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.37 warning: security/nss/lib/jar/jarfile.c:328:45 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.37 warning: security/nss/lib/jar/jarfile.c:580:21 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/jar/jarfile.c:704:55 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned int' 18:39.37 warning: security/nss/lib/jar/jarfile.c:770:59 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned int' 18:39.37 warning: security/nss/lib/jar/jarfile.c:833:53 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/debug_module.c:352:9 [-Wsign-compare] comparison of integers of different signs: 'PRLogModuleLevel' (aka 'enum PRLogModuleLevel') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/debug_module.c:356:9 [-Wsign-compare] comparison of integers of different signs: 'PRLogModuleLevel' (aka 'enum PRLogModuleLevel') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11kea.c:40:30 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11merge.c:75:30 [-Wsign-compare] comparison of integers of different signs: 'int' and 'CK_ULONG' (aka 'unsigned long') 18:39.37 warning: security/nss/lib/pk11wrap/pk11merge.c:76:44 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11merge.c:124:32 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11obj.c:1251:29 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11obj.c:1883:72 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:464:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:467:36 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:488:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:489:31 [-Wsign-compare] comparison of integers of different signs: 'int' and 'const unsigned int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:553:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:556:48 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:589:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:1417:20 [-Wsign-compare] comparison of integers of different signs: 'CK_SLOT_ID' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pars.c:1454:20 [-Wsign-compare] comparison of integers of different signs: 'CK_SLOT_ID' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pk12.c:534:31 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pqg.c:92:21 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pqg.c:302:26 [-Wsign-compare] comparison of integers of different signs: 'const unsigned int' and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11pqg.c:319:26 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:538:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:573:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:960:17 [-Wsign-compare] comparison of integers of different signs: 'CK_KEY_TYPE' (aka 'unsigned long') and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:975:21 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:1928:37 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:1976:37 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:2145:29 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11skey.c:2179:34 [-Wstring-conversion] implicit conversion turns string literal into bool: 'char [12]' to '_Bool' 18:39.37 warning: security/nss/lib/pk11wrap/pk11slot.c:1097:24 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.37 warning: security/nss/lib/pk11wrap/pk11slot.c:1400:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11slot.c:2522:22 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.37 warning: security/nss/lib/pk11wrap/pk11slot.c:2527:22 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/pk11wrap/pk11util.c:998:15 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/pk11wrap/pk11util.c:1483:17 [-Wsign-compare] comparison of integers of different signs: 'CK_SLOT_ID' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/pkcs7/certread.c:321:25 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/pkcs7/certread.c:322:29 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/pkcs7/certread.c:405:15 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/pkcs7/certread.c:433:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/pkcs7/p7local.c:666:34 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.38 warning: security/nss/lib/pkcs7/p7local.c:726:30 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.38 warning: security/nss/lib/smime/cmscipher.c:651:34 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.38 warning: security/nss/lib/smime/cmscipher.c:711:30 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'int' 18:39.38 warning: security/nss/lib/softoken/legacydb/pcertdb.c:4852:13 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/softoken/lgglue.c:44:26 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.38 warning: security/nss/lib/softoken/pkcs11.c:2390:21 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.38 warning: security/nss/lib/softoken/pkcs11.c:4458:73 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/softoken/pkcs11c.c:6876:31 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/softoken/sftkdb.c:341:69 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/softoken/sftkdb.c:1621:70 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/softoken/sftkpwd.c:900:38 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/softoken/sftkpwd.c:925:38 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/softoken/sftkpwd.c:984:38 [-Wsign-compare] comparison of integers of different signs: 'CK_ULONG' (aka 'unsigned long') and 'int' 18:39.38 warning: security/nss/lib/ssl/dtlscon.c:455:48 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.38 warning: security/nss/lib/ssl/dtlscon.c:740:26 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned int' 18:39.38 warning: security/nss/lib/ssl/dtlscon.c:926:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:4355:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'PRUint32' (aka 'unsigned int') 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:5254:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'const unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:5421:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'const unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:6791:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'const unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:7125:19 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:8069:27 [-Wsign-compare] comparison of integers of different signs: 'int' and 'const unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:8775:23 [-Wsign-compare] comparison of integers of different signs: 'int' and 'const unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:9147:16 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:9689:33 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.38 warning: security/nss/lib/ssl/ssl3con.c:10904:37 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/ssl3exthandle.c:2158:18 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/ssl3exthandle.c:2273:18 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/ssl3exthandle.c:2340:28 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/ssl3exthandle.c:2354:25 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/ssl3exthandle.c:2406:18 [-Wsign-compare] comparison of integers of different signs: 'PRUint32' (aka 'unsigned int') and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/ssl/sslsnce.c:684:31 [-Wstring-conversion] implicit conversion turns string literal into bool: 'char [28]' to '_Bool' 18:39.38 warning: security/nss/lib/ssl/sslsnce.c:711:31 [-Wstring-conversion] implicit conversion turns string literal into bool: 'char [28]' to '_Bool' 18:39.38 warning: security/nss/lib/ssl/sslsnce.c:1847:30 [-Wsign-compare] comparison of integers of different signs: 'PRInt16' (aka 'short') and 'unsigned int' 18:39.38 warning: security/nss/lib/ssl/sslsock.c:1491:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.38 warning: security/nss/lib/ssl/sslsock.c:3191:30 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/nss/lib/ssl/sslsock.c:3204:66 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.38 warning: security/nss/lib/ssl/sslsock.c:3223:20 [-Wsign-compare] comparison of integers of different signs: 'const PRInt32' (aka 'const int') and 'unsigned long' 18:39.38 warning: security/nss/lib/ssl/sslsock.c:3226:27 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'unsigned long' 18:39.38 warning: security/nss/lib/ssl/sslsock.c:3231:54 [-Wsign-compare] comparison of integers of different signs: 'unsigned long' and 'int' 18:39.38 warning: security/nss/lib/ssl/tls13con.c:1650:33 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned int' 18:39.38 warning: security/nss/lib/ssl/tls13con.c:4112:31 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'PRUint32' (aka 'unsigned int') 18:39.38 warning: security/nss/lib/ssl/tls13con.c:4117:15 [-Wsign-compare] comparison of integers of different signs: 'PRInt32' (aka 'int') and 'PRUint32' (aka 'unsigned int') 18:39.38 warning: security/nss/lib/ssl/tls13con.c:4377:23 [-Wsign-compare] comparison of integers of different signs: 'unsigned int' and 'PRInt32' (aka 'int') 18:39.38 warning: security/nss/lib/util/pkcs11uri.c:247:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.38 warning: security/nss/lib/util/pkcs11uri.c:252:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.38 warning: security/nss/lib/util/pkcs11uri.c:254:19 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.38 warning: security/nss/lib/util/pkcs11uri.c:259:17 [-Wsign-compare] comparison of integers of different signs: 'int' and 'size_t' (aka 'unsigned long') 18:39.38 warning: security/nss/lib/util/utilpars.c:188:30 [-Wsign-compare] comparison of integers of different signs: 'int' and 'unsigned long' 18:39.38 warning: security/sandbox/chromium/base/time/time.h:690:37 [-Wcomma] possible misuse of comma operator here 18:39.38 warning: toolkit/components/protobuf/src/google/protobuf/stubs/strutil.cc:313:8 [-Wcomma] possible misuse of comma operator here 18:39.38 warning: toolkit/components/protobuf/src/google/protobuf/wire_format.cc:60:12 [-Wunused-function] unused function 'GetEnumNumber' 18:39.38 warning: toolkit/crashreporter/google-breakpad/src/processor/stackwalker_mips.cc:60:9 [-Wtautological-constant-out-of-range-compare] comparison of constant 18446744073709551615 with expression of type 'bool' is always false 18:39.38 warning: toolkit/crashreporter/google-breakpad/src/processor/stackwalker_mips.cc:68:66 [-Wtautological-constant-out-of-range-compare] comparison of constant 4294967295 with expression of type 'bool' is always false 18:39.38 warning: /usr/include/features.h:148:3 [-W#warnings] "_BSD_SOURCE and _SVID_SOURCE are deprecated, use _DEFAULT_SOURCE"

To the majority of our developers, most of these happen in code that is read only for them for all intents and purposes. This currently only serves to make it more difficult to find compiler errors when they occur. FWIW people are suggesting workarounds like prefixing CC and CXX with chronic to suppress stdout output when the compiler finishes with a success error code which ironically defeats your original purpose of increases the visibility of compiler warnings, and once enough developers end up with |export CC="chronic clang"| etc in their mozconfigs the effective outcome of this change will be fewer warning fixes, I'm afraid.


What was the motivation behind this change? Was there a complaint from a significant number of developers about it being difficult fixing compiler warnings grepping for things like "warning:" or using ./mach warnings-list? Was the feedback from developers who have the use case of writing code and fix compiler errors in such code taken into account?
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to