This is the mail archive of the
binutils@sources.redhat.com
mailing list for the binutils project.
Re: [RFA] Eliminate warnings about snprintf declaration
- From: Daniel Jacobowitz <drow at false dot org>
- To: Ian Lance Taylor <ian at airs dot com>
- Cc: Eli Zaretskii <eliz at gnu dot org>, gcc-patches at gcc dot gnu dot org,gdb-patches at sourceware dot org, binutils at sourceware dot org
- Date: Sun, 15 May 2005 13:04:35 -0400
- Subject: Re: [RFA] Eliminate warnings about snprintf declaration
- References: <01c55708$Blat.v2.4$cfc9f040@zahav.net.il> <20050512154716.GA3513@nevyn.them.org> <01c5570e$Blat.v2.4$1c533160@zahav.net.il> <20050512162453.GA5180@nevyn.them.org> <01c55732$Blat.v2.4$e9bd3640@zahav.net.il> <20050512205103.GB13519@nevyn.them.org> <01c5586c$Blat.v2.4$a7dfc720@zahav.net.il> <20050514141204.GA10684@nevyn.them.org> <01c558a0$Blat.v2.4$2b06ba40@zahav.net.il> <m364xkihbc.fsf@gossamer.airs.com>
On Sun, May 15, 2005 at 12:29:11PM -0400, Ian Lance Taylor wrote:
> "Eli Zaretskii" <eliz@gnu.org> writes:
>
> > include/ChangeLog:
> > 2005-05-14 Eli Zaretskii <eliz@gnu.org>
> >
> > * libiberty.h: (snprintf) [!HAVE_DECL_SNPRINTF]: Declare if
> > needed.
> > (vsnprintf) [!HAVE_DECL_VSNPRINTF]: Declare if needed.
> >
> >
> > bfd/ChangeLog:
> > 2005-05-14 Eli Zaretskii <eliz@gnu.org>
> >
> > * configure.in: Add snprintf and vsnprintf to AC_CHECK_DECLS.
>
> These patches are approved, plus the change to libiberty/configure.ac
> which needs the obvious ChangeLog entry. Thanks.
Eli, please mention regenerating configure in the ChangeLog.
> > gdb/ChangeLog:
> > 2005-05-14 Eli Zaretskii <eliz@gnu.org>
> >
> > * configure.ac: Add snprintf and vsnprintf to AC_CHECK_DECLS.
>
> I can't approve this one, but it looks fine to me.
Eli can approve it himself; or I will :-) Same comment as above
applies; please mention configure.
--
Daniel Jacobowitz
CodeSourcery, LLC