This is the mail archive of the mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Warning roundup - ld/

[Figures... as soon as I decide things are obvious, I start making
mistakes.  Apologies.]

On Sun, Feb 10, 2002 at 11:16:08AM -0800, Ian Lance Taylor wrote:
> Daniel Jacobowitz <> writes:
> > +#ifndef HAVE_STDLIB_H
> > +/* If we have <stdlib.h>, assume it defines strtoul.  */
> >  /* Omit args to avoid the possibility of clashing with a system header
> >     that might disagree about consts.  */
> >  unsigned long strtoul ();
> > +#endif
> This is not a safe assumption, as I believe some versions of SunOS
> have <stdlib.h> but do not have strtoul().

I checked as far back as I could, which only went back to early Solaris
and not SunOS 4.

> However, I don't know lexsup.c is bothering to declare strtoul() at
> all.

Shall I:
 - just kill the declaration?  Presumably somewhere we aren't
  picking it up, which is why it was present.
 - Leave my patch - even if it is not a safe assumption it is just as safe
  as not defining it at all.

Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]