This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Warnings patrol
"Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:
> > If and only if someone then complains about the missing declaration,
> > then we can add a conditional declaration as
> >
> > extern char *ldgetname (LDFILE *, GCC_SYMENT *);
> >
> > which should work okay with both the Tru64 and AIX definitions.
> > zw
>
> I'm looking in my crystal ball and predicting that somebody's system
> declaration of ldgetname() delcares one or both of these pointers
> "const" and somebody else's doesn't. :-)
>
> IMHO if we're going to add the decl we should still do the configure
> checks. But I agree we should change the existing decl to match what
> you wrote above.
My point seems to have gone past everyone. I was suggesting that,
instead of adding yet another configure check, we simply rely on
<ldfcn.h> to declare this routine. Only if this provokes an actual
report of a problem should we put in a configure check and conditional
declaration.
zw