On Thu, May 25, 2017 at 5:31 AM, Ehsan Akhgari <ehsan.akhg...@gmail.com> wrote:
> 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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > ICEr/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/n > rappkit/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/n > rappkit/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/n > rappkit/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/n > rappkit/src/registry/registry_local.c:885:9 [-Wmacro-redefined] > 'NRREGGET' macro redefined > 18:39.36 warning: media/mtransport/third_party/n > rappkit/src/registry/registry_local.c:978:9 [-Wmacro-redefined] > 'NRREGSET' macro redefined > 18:39.36 warning: media/mtransport/third_party/n > rappkit/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/n > rappkit/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/n > rappkit/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/n > rappkit/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/comm > on_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/modu > les/audio_coding/neteq/background_noise.h:98:22 [-Wcomma] possible misuse > of comma operator here > 18:39.36 warning: media/webrtc/trunk/webrtc/modu > les/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/modu > les/audio_processing/aecm/aecm_defines.h:23:9 [-Wmacro-redefined] > 'PART_LEN2' macro redefined > 18:39.36 warning: media/webrtc/trunk/webrtc/modu > les/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/modu > les/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/modu > les/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/modu > les/desktop_capture/differ_unittest.cc:187:22 [-Wcomma] possible misuse > of comma operator here > 18:39.36 warning: media/webrtc/trunk/webrtc/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/rtp_rtcp/source/rtp_format_h264.cc:47:6 [-Wunused-function] unused > function 'VerifyStapANaluLengths' > 18:39.36 warning: media/webrtc/trunk/webrtc/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/modu > les/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/vide > o_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/voic > e_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/voic > e_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/voic > e_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/voic > e_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/voic > e_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/di > st/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/di > st/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/di > st/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/sr > c/google/protobuf/stubs/strutil.cc:313:8 [-Wcomma] possible misuse of > comma operator here > 18:39.38 warning: toolkit/components/protobuf/sr > c/google/protobuf/wire_format.cc:60:12 [-Wunused-function] unused > function 'GetEnumNumber' > 18:39.38 warning: toolkit/crashreporter/google-b > reakpad/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-b > reakpad/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? > The motification was that relevant compiler warnings are flying under the radar and are detected too late in the development process - such as when pushing to Try. We want to catch problems as early in the development cycle as possible to reduce the end-to-end times for patch development. There have been complaints from developers over the years about this. What specifically motivated me to make this change recently was finding a pattern of feature requests during bug triage that all seemed to point to improving visibility of compiler warnings as a potential solution. I fully acknowledge that the spew of warnings at the end of the build is annoying. This was expected to cause temporary pain. The hope was that this would spur people into fixing the warnings. There has been some movement there. ICU warnings were suppressed, for example. And, there has been talk in this thread (and presumably elsewhere) about fixing other large offenders. So on that front, I'd say this change was successful. But the pace of progress hasn't been terrific and the status quo is annoying. This thread is also sapping up people's time. I've prepared a patch to tweak the behavior that I think strikes a compromise. I'll file a bug to track the review shortly. _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
