warning:Mime-Version: 1.0

[Hal Murray]

devel at ntpsec.org said:
> Oh.  Gary's builds must be using "./waf configure --enable-warnings".  That
> enables "-Wswitch-default" and other extra checks where available.  I'll
> check the implications of enabling that for all jobs and make also enabling
> "-Werror" so that GitLab builds will fail in a noticable way before end-users
> see warnings.

There was one of those that generated a lot of noise so we didn't enable it by 
default.

I just tried --enable-warning on my collection.  Only two warnings:

We can easily fix this one.
../../ntpd/ntp_monitor.c:136: warning: inlining failed in call to 
‘mon_reclaim_entry’: call is unlikely and code size would grow
../../ntpd/ntp_monitor.c:434: warning: called from here
../../ntpd/ntp_monitor.c:136: warning: inlining failed in call to 
‘mon_reclaim_entry’: call is unlikely and code size would grow
../../ntpd/ntp_monitor.c:447: warning: called from here

This one looks more complicated:
ntp_parser.tab.c:1469:3: warning: switch missing default case 
[-Wswitch-default]
It only generates warnings on a few systems.  I don't see why not on the 
others.   ??


-- 
These are my opinions.  I hate spam.