warnings from recent git pull
Gary E. Miller
gem at rellim.com
Fri May 26 01:21:29 UTC 2017
Yo Hal!
On Thu, 25 May 2017 18:03:10 -0700
Hal Murray <hmurray at megapathdsl.net> wrote:
> > Fixes pushed, give it a try.
>
> Looks good, but some builds still in the pipeline and I'm about to
> leave. Assume OK unless I get back to you.
I'll watch buildbot.
> >> From Fedora and many/most other systems:
> > Not 'most' from where I sit.
>
> Did you try tests/option-tester.sh? A lot of that stuff doesn't
> happen unless you use the right configure options.
This was related to things waf detected about the OS, not to config
options.
> >> ../../tests/libparse/binio.c:130:23: warning: declaration of 'exp'
> >> shadows a global declaration
> > As I suspected, a NetBSD name collision. Fix pushed.
>
> OpenBSD gets the exp/shadow warnings. So does an old CentOS.
Still? I no longer see any 'exp' in binio. Got the log output?
> I don't understand why it doesn't happen on all systems.
Because netBSD defines a macro called 'exp()', but Linux does not.
> ntp_fp.h
> includes math.h which defines exp.
Only in some OS. And nothing that includes ntp_fp.h defines any
variable called exp. Except for whatever one I missed.
> Is there some magic that drops
> that warning if you don't use the symbol from the header in some
> other part of the code being compiled?
#ifdef exp
# undef exp
#endif
But easiest just to not create a variable called exp.
> Maybe there were too many false positives like that one?
Huh?
RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
gem at rellim.com Tel:+1 541 382 8588
Veritas liberabit vos. -- Quid est veritas?
"If you can’t measure it, you can’t improve it." - Lord Kelvin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ntpsec.org/pipermail/devel/attachments/20170525/cfa3b8ad/attachment.bin>
More information about the devel
mailing list