From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gdb-patches-return-39470-listarch-gdb-patches=sources.redhat.com@sources.redhat.com>
Received: (qmail 2627 invoked by alias); 15 May 2005 17:04:45 -0000
Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm
Precedence: bulk
List-Subscribe: <mailto:gdb-patches-subscribe@sources.redhat.com>
List-Archive: <http://sources.redhat.com/ml/gdb-patches/>
List-Post: <mailto:gdb-patches@sources.redhat.com>
List-Help: <mailto:gdb-patches-help@sources.redhat.com>, <http://sources.redhat.com/ml/#faqs>
Sender: gdb-patches-owner@sources.redhat.com
Received: (qmail 2561 invoked from network); 15 May 2005 17:04:42 -0000
Received: from unknown (HELO nevyn.them.org) (66.93.172.17)
  by sourceware.org with SMTP; 15 May 2005 17:04:42 -0000
Received: from drow by nevyn.them.org with local (Exim 4.50)
	id 1DXMXb-0000BC-Sh; Sun, 15 May 2005 13:04:36 -0400
Date: Sun, 15 May 2005 17:15:00 -0000
From: Daniel Jacobowitz <drow@false.org>
To: Ian Lance Taylor <ian@airs.com>
Cc: Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
	gdb-patches@sourceware.org, binutils@sourceware.org
Subject: Re: [RFA] Eliminate warnings about snprintf declaration
Message-ID: <20050515170435.GB11855@nevyn.them.org>
Mail-Followup-To: Ian Lance Taylor <ian@airs.com>,
	Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
	gdb-patches@sourceware.org, binutils@sourceware.org
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>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <m364xkihbc.fsf@gossamer.airs.com>
User-Agent: Mutt/1.5.8i
X-SW-Source: 2005-05/txt/msg00373.txt.bz2

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