On Fri, May 26, 2017 at 1:50 AM, Gregory Szorc <g...@mozilla.com> wrote:
> 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. > Yes, I'm aware of this, as I was one of the people who requested it. The point I was making is that it was an example of something that, like this, was a change to the mach output format that some people may have appreciated but also disrupted people's existing work flows. 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. > I'm not saying that some people don't like it. I'm saying that others don't and because one more or less now needs to use mach, changes which are improvements for some and regressions for others need to be made with extreme care. -Ekr > >> >> 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
