[PATCH] libiberty: add HAVE_CONFIG_H check to all files (lacking it already)

Ian Lance Taylor iant@google.com
Tue Aug 17 14:01:00 GMT 2010


Mike Frysinger <vapier@gentoo.org> writes:

> On Tue, Aug 17, 2010 at 9:15 AM, Ian Lance Taylor wrote:
>> No, those files were copied from projects like gnulib, which exist to
>> provide files to be included in other programs.
>
> looking closer, i'm not sure that's true.  there are plenty of files
> labeled "This file is part of libiberty." that have HAVE_CONFIG_H
> protection.  what really strikes me though is that some files which
> have this protection require other files which do not.  which again is
> how i got here in the first place.

In that case I would personally prefer that the #ifdef HAVE_CONFIG_H be
removed from those files, since it should not be needed.

Ian



More information about the Binutils mailing list