This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: [RFA] Fix compilation warning in libiberty.h
- From: Daniel Jacobowitz <drow at false dot org>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: binutils at sources dot redhat dot com
- Date: Fri, 18 Mar 2005 13:01:09 -0500
- Subject: Re: [RFA] Fix compilation warning in libiberty.h
- References: <20050318175552.GO1156@adacore.com>
On Fri, Mar 18, 2005 at 09:55:52AM -0800, Joel Brobecker wrote:
> Hello,
>
> I noticed that the compiler emits a warning when building bfd.c
> in libiberty.h. The compiler we are using is GNAT Pro based on
> GCC 3.4.4. Here is the warning we get:
>
> gcc -DHAVE_CONFIG_H -I. -I.././bfd -I. -D_GNU_SOURCE -I. -I.././bfd -I.././bfd/../include -I.././bfd/../intl -I../intl -W -Wall -Wstrict-prototypes -Wmissing-prototypes -Werror -g -O2 -c bfd.c -o bfd.o
> In file included from bfd.c:209:
> .././bfd/../include/libiberty.h:80: warning: function declaration isn't a prototype
>
> The problem is that this breaks the build because of the -Werror.
>
> I think the problem is that the prototype is missing a ``void'' keyword.
> This fixes the warning. The trouble I had was whether I wanted to use
> the PARAMS macro or not. I seem to remember that we wanted to get away
> from them, but then the rest of the file uses it. I kept it for now,
> for better consistency.
>
> Joel Brobecker <brobecker@adacore.com>
>
> * include/libiberty.h,v (basename): Fix prototype definition.
>
> Tested by building GDB on sparc-solaris 2.8.
> OK to apply?
No, this is wrong; that's not supposed to be a prototype! You want to
be in the other branch of the #if; fix that instead.
--
Daniel Jacobowitz
CodeSourcery, LLC