On Thu, May 25, 2017 at 7:43 AM, Eric Rescorla <e...@rtfm.com> wrote:
> I'd like to second Ehsan's point, but also expand upon it into a more > general observation. > > As it becomes progressively more difficult to build Firefox without mach, > it becomes > increasingly important that mach respect people's workflows. For those of > us who > were comfortable with make and the behavior of having relatively > unfiltered access > to what the build system is doing, it would be great if mach could have > some set of > flags to preserve that view (cf. the flags to remove the timestamps so > that emacs > compiler mode works). > `mach help build` says the answer to this particular problem is `mach --log-no-times build`. And if you really don't like the time prefixing, set the MACH_NO_WRITE_TIMES environment variable. FWIW the times are there to give developers a sense of progress. Teaching the machines to estimate progress/ETA for us is a lot more work than feeding a low-noise signal into the superb pattern recognition apparatus that is the human brain. > > On Thu, May 25, 2017 at 8:31 PM, 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? >> >> _______________________________________________ >> dev-platform mailing list >> dev-platform@lists.mozilla.org >> https://lists.mozilla.org/listinfo/dev-platform >> > > _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
