src/binutils ChangeLog coffdump.c nlmconv.c nl ...
Daniel Jacobowitz
drow@mvista.com
Sun Feb 10 11:24:00 GMT 2002
On Sun, Feb 10, 2002 at 11:13:05AM -0800, Ian Lance Taylor wrote:
> drow@sources.redhat.com writes:
>
> > * nlmconv.c: Add PARAMS ((const time_t *)) to localtime
> > prototype. Prototype main.
> > * nlmheader.y: Add PARAMS ((int)) to strerror prototype.
>
> Please don't make these changes. The localtime change in particular
> will cause the binutils to fail to compile on some platforms, those on
> which localtime is declare in the system header files to take time_t *
> rather than const time_t *.
>
> Never use a full prototype when declaring functions provided the
> system. It can only cause conflicts and, except in the case of a
> prototype which uses an ellipsis, can never help. If you feel that
> you must use a full prototype, you must add a configure test to
> confirm that it will compile correctly.
I assume you're referring to localtime here; strerror was already
handled this way in nlmconv.c, directly above where I added a prototype
for localtime.
In this case, should we turn off the warning I was fixing,
-Wstrict-prototypes, or should I add configury for localtime's
declaration?
--
Daniel Jacobowitz Carnegie Mellon University
MontaVista Software Debian GNU/Linux Developer
More information about the Binutils
mailing list