Re: switch missing default case

2019-04-08 Thread Hal Murray via devel
Gary said: > I fixed the libjsmn missing default one. Thanks. And thanks to whomever fixed the MacOS glitch. -- These are my opinions. I hate spam. ___ devel mailing list devel@ntpsec.org http://lists.ntpsec.org/mailman/listinfo/devel

Re: switch missing default case

2019-04-08 Thread Gary E. Miller via devel
Yo Hal! On Sun, 07 Apr 2019 21:08:44 -0700 Hal Murray via devel wrote: > > I'm unaware of any we can't fix, except the bison one. > > There are several others. > > From old CentOS: > From NetBSD on a Raspbery Pi: > From old (but still supported) NetBSD: Don't have insight on those OS. I fi

Re: switch missing default case

2019-04-07 Thread Hal Murray via devel
> I'm unaware of any we can't fix, except the bison one. There are several others. >From old CentOS: > ntp_parser.tab.c:389:6: warning: "YYENABLE_NLS" is not defined > ntp_parser.tab.c:1323:6: warning: "YYLTYPE_IS_TRIVIAL" is not defined >From NetBSD on a Raspbery Pi: > /usr/pkg/lib/libpython2.

Re: switch missing default case

2019-04-07 Thread Gary E. Miller via devel
Yo Hal! On Sun, 07 Apr 2019 16:29:48 -0700 Hal Murray via devel wrote: > > New warning on arm64: > > Also happens on Fedora, both 64 and 32 bit. Easy to fix. If no one beats me to it I'll fix Monday. > Is it reasonable to fix the CI system to complain about warnings > except for a (hopeful

Re: switch missing default case

2019-04-07 Thread Hal Murray via devel
> New warning on arm64: Also happens on Fedora, both 64 and 32 bit. Is it reasonable to fix the CI system to complain about warnings except for a (hopefully short) list of known ones that we can't fix? -- These are my opinions. I hate spam. ___

✘switch missing default case

2019-04-07 Thread Gary E. Miller via devel
Yo All! New warning on arm64: [ 99/144] Compiling ntpd/ntp_loopfilter.c In file included from ../../ntpd/refclock_gpsd.c:78: ../../ntpd/../libjsmn/jsmn.c: In function ‘jsmn_parse_primitive’: ../../ntpd/../libjsmn/jsmn.c:43:3: warning: switch missing default case [-Wswitch-default] switch (js

✘warning: switch missing default case

2018-06-17 Thread Gary E. Miller via devel
Yo All! I'm seeing some warnings on NTPsec git head with Gentoo stable on RasPi. [ 27/132] Compiling libntp/getopt.c ../../libntp/authkeys.c: In function ‘alloc_auth_info’: ../../libntp/authkeys.c:290:2: warning: switch missing default case [-Wswitch-de fault] switch