* Re: [RFA] Eliminate warnings about snprintf declaration
[not found] ` <20050514141204.GA10684@nevyn.them.org>
@ 2005-05-14 19:02 ` Eli Zaretskii
2005-05-15 16:52 ` Ian Lance Taylor
0 siblings, 1 reply; 11+ messages in thread
From: Eli Zaretskii @ 2005-05-14 19:02 UTC (permalink / raw)
To: gcc-patches; +Cc: gdb-patches, binutils
> Date: Sat, 14 May 2005 10:12:04 -0400
> From: Daniel Jacobowitz <drow@false.org>
> Cc: gcc-patches@gcc.gnu.org, gdb-patches@sources.redhat.com
>
> > Actually, I found that a simpler patch to include/libiberty.h (below)
> > is all that is needed to fix all the warnings about snprintf and
> > vsnprintf. libiberty.h already does that for asprintf and vasprintf,
> > so I think we can use the same method for snprintf and vsnprintf; no
> > need to change any configure.ac files.
>
> > 2005-05-14 Eli Zaretskii <eliz@gnu.org>
> >
> > * libiberty.h: (snprintf) [!HAVE_DECL_SNPRINTF]: Declare if
> > needed.
> > (vsnprintf) [!HAVE_DECL_VSNPRINTF]: Declare if needed.
>
> Note that this will only work if there are no systems which declare
> snprintf with an incorrect prototype. For instance, a missing const on
> the format string in a system header would now generate an error. For
> asprintf this isn't an issue, because it's not a standard function;
> it's only provided by glibc.
Actually, what I wrote was completely bogus: without adding these
functions to AC_CHECK_DECLS, the respective HAVE_DECL_* macros will
never get defined, so what I suggested is equivalent to including
these prototypes unconditionally, which is certainly not what I meant.
Sorry.
Here's a revised patch, now together with the configury stuff.
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.
gdb/ChangeLog:
2005-05-14 Eli Zaretskii <eliz@gnu.org>
* configure.ac: Add snprintf and vsnprintf to AC_CHECK_DECLS.
Index: include/libiberty.h
===================================================================
RCS file: /cvs/src/src/include/libiberty.h,v
retrieving revision 1.44
diff -u -r1.44 libiberty.h
--- include/libiberty.h 12 May 2005 20:00:35 -0000 1.44
+++ include/libiberty.h 14 May 2005 16:09:49 -0000
@@ -531,6 +531,16 @@
ATTRIBUTE_PRINTF(2,0);
#endif
+#if defined(HAVE_DECL_SNPRINTF) && !HAVE_DECL_SNPRINTF
+/* Like sprintf but prints at most N characters. */
+extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
+#endif
+
+#if defined(HAVE_DECL_VSNPRINTF) && !HAVE_DECL_VSNPRINTF
+/* Like vsprintf but prints at most N characters. */
+extern int vsnprintf (char *, size_t, const char *, va_list);
+#endif
+
#define ARRAY_SIZE(a) (sizeof (a) / sizeof ((a)[0]))
/* Drastically simplified alloca configurator. If we're using GCC,
Index: bfd/configure.in
===================================================================
RCS file: /cvs/src/src/bfd/configure.in,v
retrieving revision 1.183
diff -u -r1.183 configure.in
--- bfd/configure.in 10 May 2005 01:27:53 -0000 1.183
+++ bfd/configure.in 14 May 2005 16:10:11 -0000
@@ -144,6 +144,8 @@
AC_CHECK_DECLS(realloc)
AC_CHECK_DECLS(stpcpy)
AC_CHECK_DECLS(strstr)
+AC_CHECK_DECLS(snprintf)
+AC_CHECK_DECLS(vsnprintf)
# If we are configured native, pick a core file support file.
COREFILE=
Index: gdb/configure.ac
===================================================================
RCS file: /cvs/src/src/gdb/configure.ac,v
retrieving revision 1.18
diff -u -r1.18 configure.ac
--- gdb/configure.ac 21 Apr 2005 05:34:33 -0000 1.18
+++ gdb/configure.ac 14 May 2005 16:10:31 -0000
@@ -418,7 +418,7 @@
AC_CHECK_DECLS([free, malloc, realloc])
AC_CHECK_DECLS([strerror, strstr])
-AC_CHECK_DECLS(getopt)
+AC_CHECK_DECLS([getopt, snprintf, vsnprintf])
# ----------------------- #
# Checks for structures. #
Index: libiberty/configure.ac
===================================================================
RCS file: /cvs/src/src/libiberty/configure.ac,v
retrieving revision 1.22
diff -u -r1.22 configure.ac
--- libiberty/configure.ac 7 May 2005 02:00:41 -0000 1.22
+++ libiberty/configure.ac 14 May 2005 16:10:46 -0000
@@ -282,7 +282,7 @@
sysconf times sbrk gettimeofday ffs snprintf vsnprintf \
pstat_getstatic pstat_getdynamic sysmp getsysinfo table sysctl wait3 wait4 \
realpath canonicalize_file_name __fsetlocking)
- AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf])
+ AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf, snprintf, vsnprintf])
AC_DEFINE(HAVE_SYS_ERRLIST, 1, [Define if you have the sys_errlist variable.])
AC_DEFINE(HAVE_SYS_NERR, 1, [Define if you have the sys_nerr variable.])
AC_DEFINE(HAVE_SYS_SIGLIST, 1, [Define if you have the sys_siglist variable.])
@@ -518,7 +518,7 @@
[AC_MSG_RESULT([no])])
AC_CHECK_FUNCS($checkfuncs)
- AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf])
+ AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf, snprintf, vsnprintf])
libiberty_NEED_DECLARATION(canonicalize_file_name)
fi
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-14 19:02 ` [RFA] Eliminate warnings about snprintf declaration Eli Zaretskii
@ 2005-05-15 16:52 ` Ian Lance Taylor
2005-05-15 17:28 ` Daniel Jacobowitz
2005-05-15 19:55 ` Eli Zaretskii
0 siblings, 2 replies; 11+ messages in thread
From: Ian Lance Taylor @ 2005-05-15 16:52 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: gcc-patches, gdb-patches, binutils
"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.
> 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.
Ian
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-15 16:52 ` Ian Lance Taylor
@ 2005-05-15 17:28 ` Daniel Jacobowitz
2005-05-15 19:48 ` Eli Zaretskii
2005-05-15 19:55 ` Eli Zaretskii
1 sibling, 1 reply; 11+ messages in thread
From: Daniel Jacobowitz @ 2005-05-15 17:28 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: Eli Zaretskii, gcc-patches, gdb-patches, binutils
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
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-15 17:28 ` Daniel Jacobowitz
@ 2005-05-15 19:48 ` Eli Zaretskii
2005-05-15 20:26 ` Daniel Jacobowitz
0 siblings, 1 reply; 11+ messages in thread
From: Eli Zaretskii @ 2005-05-15 19:48 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: gcc-patches, gdb-patches, binutils
> Date: Sun, 15 May 2005 13:04:35 -0400
> From: Daniel Jacobowitz <drow@false.org>
> Cc: Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
> gdb-patches@sourceware.org, binutils@sourceware.org
>
> > > 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.
I didn't send patches for the (regenerated) configure and config.in.
Should I do that, given that I cannot commit them anyway?
> > > 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 :-)
True; but given the problems in my original patch, I prefered to get
this by you-all.
> Same comment as above applies; please mention configure.
(And config.in, right?) Will do.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-15 16:52 ` Ian Lance Taylor
2005-05-15 17:28 ` Daniel Jacobowitz
@ 2005-05-15 19:55 ` Eli Zaretskii
1 sibling, 0 replies; 11+ messages in thread
From: Eli Zaretskii @ 2005-05-15 19:55 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: gcc-patches, gdb-patches, binutils
> Cc: gcc-patches@gcc.gnu.org, gdb-patches@sourceware.org,
> binutils@sourceware.org
> From: Ian Lance Taylor <ian@airs.com>
> Date: 15 May 2005 12:29:11 -0400
>
> "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.
Thank you.
Could someone please commit these? I don't have write access to bfd
and gcc repositories.
TIA
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-15 19:48 ` Eli Zaretskii
@ 2005-05-15 20:26 ` Daniel Jacobowitz
2005-05-20 12:55 ` Eli Zaretskii
0 siblings, 1 reply; 11+ messages in thread
From: Daniel Jacobowitz @ 2005-05-15 20:26 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: Ian Lance Taylor, gcc-patches, gdb-patches, binutils
On Sun, May 15, 2005 at 10:40:59PM +0300, Eli Zaretskii wrote:
> > Date: Sun, 15 May 2005 13:04:35 -0400
> > From: Daniel Jacobowitz <drow@false.org>
> > Cc: Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
> > gdb-patches@sourceware.org, binutils@sourceware.org
> >
> > > > 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.
>
> I didn't send patches for the (regenerated) configure and config.in.
> Should I do that, given that I cannot commit them anyway?
No, you don't need to. If you mention them in the ChangeLog, that'll
serve as a hint to whoever commits the patch which files need to be
regenerated.
> > Same comment as above applies; please mention configure.
>
> (And config.in, right?) Will do.
Good catch. Yes, thank you.
--
Daniel Jacobowitz
CodeSourcery, LLC
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-15 20:26 ` Daniel Jacobowitz
@ 2005-05-20 12:55 ` Eli Zaretskii
2005-05-28 11:38 ` Eli Zaretskii
0 siblings, 1 reply; 11+ messages in thread
From: Eli Zaretskii @ 2005-05-20 12:55 UTC (permalink / raw)
To: Ian Lance Taylor; +Cc: gcc-patches, gdb-patches, binutils
> Date: Sun, 15 May 2005 15:55:23 -0400
> From: Daniel Jacobowitz <drow@false.org>
> Cc: Ian Lance Taylor <ian@airs.com>, gcc-patches@gcc.gnu.org,
> gdb-patches@sourceware.org, binutils@sourceware.org
>
> On Sun, May 15, 2005 at 10:40:59PM +0300, Eli Zaretskii wrote:
> > > Date: Sun, 15 May 2005 13:04:35 -0400
> > > From: Daniel Jacobowitz <drow@false.org>
> > > Cc: Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
> > > gdb-patches@sourceware.org, binutils@sourceware.org
> > >
> > > > > 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.
> >
> > I didn't send patches for the (regenerated) configure and config.in.
> > Should I do that, given that I cannot commit them anyway?
>
> No, you don't need to. If you mention them in the ChangeLog, that'll
> serve as a hint to whoever commits the patch which files need to be
> regenerated.
>
> > > Same comment as above applies; please mention configure.
> >
> > (And config.in, right?) Will do.
>
> Good catch. Yes, thank you.
Okay, here's the revised patch with revised ChangeLog entries. I will
commit the GDB part shortly, but I need someone else to commit the
patches in bfd/, include/, and libiberty/. TIA
libiberty/ChangeLog:
2005-05-20 Eli Zaretskii <eliz@gnu.org>
* configure.ac: Add snprintf and vsnprintf to AC_CHEK_DECLS.
* config.in, configure: Regenerate.
include/ChangeLog:
2005-05-20 Eli Zaretskii <eliz@gnu.org>
* libiberty.h: (snprintf) [!HAVE_DECL_SNPRINTF]: Declare if
needed.
(vsnprintf) [!HAVE_DECL_VSNPRINTF]: Declare if needed.
gdb/ChangeLog:
2005-05-20 Eli Zaretskii <eliz@gnu.org>
* configure.ac: Add snprintf and vsnprintf to AC_CHECK_DECLS.
* config.in, configure: Regenerate.
bfd/ChangeLog:
2005-05-20 Eli Zaretskii <eliz@gnu.org>
* configure.in: Add snprintf and vsnprintf to AC_CHECK_DECLS.
* config.in, configure: Regenerate.
Index: bfd/configure.in
===================================================================
RCS file: /cvs/src/src/bfd/configure.in,v
retrieving revision 1.183
diff -u -r1.183 configure.in
--- bfd/configure.in 10 May 2005 01:27:53 -0000 1.183
+++ bfd/configure.in 20 May 2005 11:16:55 -0000
@@ -144,6 +144,8 @@
AC_CHECK_DECLS(realloc)
AC_CHECK_DECLS(stpcpy)
AC_CHECK_DECLS(strstr)
+AC_CHECK_DECLS(snprintf)
+AC_CHECK_DECLS(vsnprintf)
# If we are configured native, pick a core file support file.
COREFILE=
cvs diff: I know nothing about gdb/configure.in
Index: include/libiberty.h
===================================================================
RCS file: /cvs/src/src/include/libiberty.h,v
retrieving revision 1.44
diff -u -r1.44 libiberty.h
--- include/libiberty.h 12 May 2005 20:00:35 -0000 1.44
+++ include/libiberty.h 20 May 2005 11:20:18 -0000
@@ -531,6 +531,16 @@
ATTRIBUTE_PRINTF(2,0);
#endif
+#if defined(HAVE_DECL_SNPRINTF) && !HAVE_DECL_SNPRINTF
+/* Like sprintf but prints at most N characters. */
+extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
+#endif
+
+#if defined(HAVE_DECL_VSNPRINTF) && !HAVE_DECL_VSNPRINTF
+/* Like vsprintf but prints at most N characters. */
+extern int vsnprintf (char *, size_t, const char *, va_list);
+#endif
+
#define ARRAY_SIZE(a) (sizeof (a) / sizeof ((a)[0]))
/* Drastically simplified alloca configurator. If we're using GCC,
Index: libiberty/configure.ac
===================================================================
RCS file: /cvs/src/src/libiberty/configure.ac,v
retrieving revision 1.22
diff -u -r1.22 configure.ac
--- libiberty/configure.ac 7 May 2005 02:00:41 -0000 1.22
+++ libiberty/configure.ac 20 May 2005 11:22:47 -0000
@@ -282,7 +282,7 @@
sysconf times sbrk gettimeofday ffs snprintf vsnprintf \
pstat_getstatic pstat_getdynamic sysmp getsysinfo table sysctl wait3 wait4 \
realpath canonicalize_file_name __fsetlocking)
- AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf])
+ AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf, snprintf, vsnprintf])
AC_DEFINE(HAVE_SYS_ERRLIST, 1, [Define if you have the sys_errlist variable.])
AC_DEFINE(HAVE_SYS_NERR, 1, [Define if you have the sys_nerr variable.])
AC_DEFINE(HAVE_SYS_SIGLIST, 1, [Define if you have the sys_siglist variable.])
@@ -518,7 +518,7 @@
[AC_MSG_RESULT([no])])
AC_CHECK_FUNCS($checkfuncs)
- AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf])
+ AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf, snprintf, vsnprintf])
libiberty_NEED_DECLARATION(canonicalize_file_name)
fi
Index: gdb/configure.ac
===================================================================
RCS file: /cvs/src/src/gdb/configure.ac,v
retrieving revision 1.18
diff -u -r1.18 configure.ac
--- gdb/configure.ac 21 Apr 2005 05:34:33 -0000 1.18
+++ gdb/configure.ac 20 May 2005 11:28:33 -0000
@@ -418,7 +418,7 @@
AC_CHECK_DECLS([free, malloc, realloc])
AC_CHECK_DECLS([strerror, strstr])
-AC_CHECK_DECLS(getopt)
+AC_CHECK_DECLS([getopt, snprintf, vsnprintf])
# ----------------------- #
# Checks for structures. #
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-20 12:55 ` Eli Zaretskii
@ 2005-05-28 11:38 ` Eli Zaretskii
2005-05-28 12:43 ` Andreas Jaeger
0 siblings, 1 reply; 11+ messages in thread
From: Eli Zaretskii @ 2005-05-28 11:38 UTC (permalink / raw)
To: gcc-patches, binutils
1-week ping. Would someone with write access to GCC and Binutils CVS
trees please commit these changes, and also regenerate config.in and
configured where the ChangeLog entries say so?
TIA
> Date: Fri, 20 May 2005 14:27:24 +0300
> From: "Eli Zaretskii" <eliz@gnu.org>
> CC: gcc-patches@gcc.gnu.org, gdb-patches@sourceware.org,
> binutils@sourceware.org
>
> > Date: Sun, 15 May 2005 15:55:23 -0400
> > From: Daniel Jacobowitz <drow@false.org>
> > Cc: Ian Lance Taylor <ian@airs.com>, gcc-patches@gcc.gnu.org,
> > gdb-patches@sourceware.org, binutils@sourceware.org
> >
> > On Sun, May 15, 2005 at 10:40:59PM +0300, Eli Zaretskii wrote:
> > > > Date: Sun, 15 May 2005 13:04:35 -0400
> > > > From: Daniel Jacobowitz <drow@false.org>
> > > > Cc: Eli Zaretskii <eliz@gnu.org>, gcc-patches@gcc.gnu.org,
> > > > gdb-patches@sourceware.org, binutils@sourceware.org
> > > >
> > > > > > 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.
> > >
> > > I didn't send patches for the (regenerated) configure and config.in.
> > > Should I do that, given that I cannot commit them anyway?
> >
> > No, you don't need to. If you mention them in the ChangeLog, that'll
> > serve as a hint to whoever commits the patch which files need to be
> > regenerated.
> >
> > > > Same comment as above applies; please mention configure.
> > >
> > > (And config.in, right?) Will do.
> >
> > Good catch. Yes, thank you.
>
> Okay, here's the revised patch with revised ChangeLog entries. I will
> commit the GDB part shortly, but I need someone else to commit the
> patches in bfd/, include/, and libiberty/. TIA
>
> libiberty/ChangeLog:
> 2005-05-20 Eli Zaretskii <eliz@gnu.org>
>
> * configure.ac: Add snprintf and vsnprintf to AC_CHEK_DECLS.
> * config.in, configure: Regenerate.
>
> include/ChangeLog:
> 2005-05-20 Eli Zaretskii <eliz@gnu.org>
>
> * libiberty.h: (snprintf) [!HAVE_DECL_SNPRINTF]: Declare if
> needed.
> (vsnprintf) [!HAVE_DECL_VSNPRINTF]: Declare if needed.
>
> gdb/ChangeLog:
> 2005-05-20 Eli Zaretskii <eliz@gnu.org>
>
> * configure.ac: Add snprintf and vsnprintf to AC_CHECK_DECLS.
> * config.in, configure: Regenerate.
>
> bfd/ChangeLog:
> 2005-05-20 Eli Zaretskii <eliz@gnu.org>
>
> * configure.in: Add snprintf and vsnprintf to AC_CHECK_DECLS.
> * config.in, configure: Regenerate.
>
>
> Index: bfd/configure.in
> ===================================================================
> RCS file: /cvs/src/src/bfd/configure.in,v
> retrieving revision 1.183
> diff -u -r1.183 configure.in
> --- bfd/configure.in 10 May 2005 01:27:53 -0000 1.183
> +++ bfd/configure.in 20 May 2005 11:16:55 -0000
> @@ -144,6 +144,8 @@
> AC_CHECK_DECLS(realloc)
> AC_CHECK_DECLS(stpcpy)
> AC_CHECK_DECLS(strstr)
> +AC_CHECK_DECLS(snprintf)
> +AC_CHECK_DECLS(vsnprintf)
>
> # If we are configured native, pick a core file support file.
> COREFILE=
> cvs diff: I know nothing about gdb/configure.in
> Index: include/libiberty.h
> ===================================================================
> RCS file: /cvs/src/src/include/libiberty.h,v
> retrieving revision 1.44
> diff -u -r1.44 libiberty.h
> --- include/libiberty.h 12 May 2005 20:00:35 -0000 1.44
> +++ include/libiberty.h 20 May 2005 11:20:18 -0000
> @@ -531,6 +531,16 @@
> ATTRIBUTE_PRINTF(2,0);
> #endif
>
> +#if defined(HAVE_DECL_SNPRINTF) && !HAVE_DECL_SNPRINTF
> +/* Like sprintf but prints at most N characters. */
> +extern int snprintf (char *, size_t, const char *, ...) ATTRIBUTE_PRINTF_3;
> +#endif
> +
> +#if defined(HAVE_DECL_VSNPRINTF) && !HAVE_DECL_VSNPRINTF
> +/* Like vsprintf but prints at most N characters. */
> +extern int vsnprintf (char *, size_t, const char *, va_list);
> +#endif
> +
> #define ARRAY_SIZE(a) (sizeof (a) / sizeof ((a)[0]))
>
> /* Drastically simplified alloca configurator. If we're using GCC,
> Index: libiberty/configure.ac
> ===================================================================
> RCS file: /cvs/src/src/libiberty/configure.ac,v
> retrieving revision 1.22
> diff -u -r1.22 configure.ac
> --- libiberty/configure.ac 7 May 2005 02:00:41 -0000 1.22
> +++ libiberty/configure.ac 20 May 2005 11:22:47 -0000
> @@ -282,7 +282,7 @@
> sysconf times sbrk gettimeofday ffs snprintf vsnprintf \
> pstat_getstatic pstat_getdynamic sysmp getsysinfo table sysctl wait3 wait4 \
> realpath canonicalize_file_name __fsetlocking)
> - AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf])
> + AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf, snprintf, vsnprintf])
> AC_DEFINE(HAVE_SYS_ERRLIST, 1, [Define if you have the sys_errlist variable.])
> AC_DEFINE(HAVE_SYS_NERR, 1, [Define if you have the sys_nerr variable.])
> AC_DEFINE(HAVE_SYS_SIGLIST, 1, [Define if you have the sys_siglist variable.])
> @@ -518,7 +518,7 @@
> [AC_MSG_RESULT([no])])
>
> AC_CHECK_FUNCS($checkfuncs)
> - AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf])
> + AC_CHECK_DECLS([basename, ffs, asprintf, vasprintf, snprintf, vsnprintf])
> libiberty_NEED_DECLARATION(canonicalize_file_name)
> fi
>
> Index: gdb/configure.ac
> ===================================================================
> RCS file: /cvs/src/src/gdb/configure.ac,v
> retrieving revision 1.18
> diff -u -r1.18 configure.ac
> --- gdb/configure.ac 21 Apr 2005 05:34:33 -0000 1.18
> +++ gdb/configure.ac 20 May 2005 11:28:33 -0000
> @@ -418,7 +418,7 @@
>
> AC_CHECK_DECLS([free, malloc, realloc])
> AC_CHECK_DECLS([strerror, strstr])
> -AC_CHECK_DECLS(getopt)
> +AC_CHECK_DECLS([getopt, snprintf, vsnprintf])
>
> # ----------------------- #
> # Checks for structures. #
>
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-28 11:38 ` Eli Zaretskii
@ 2005-05-28 12:43 ` Andreas Jaeger
2005-05-28 21:48 ` Eli Zaretskii
2005-05-30 17:50 ` DJ Delorie
0 siblings, 2 replies; 11+ messages in thread
From: Andreas Jaeger @ 2005-05-28 12:43 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: gcc-patches, binutils
[-- Attachment #1: Type: text/plain, Size: 683 bytes --]
Eli Zaretskii <eliz@gnu.org> writes:
> 1-week ping. Would someone with write access to GCC and Binutils CVS
> trees please commit these changes, and also regenerate config.in and
> configured where the ChangeLog entries say so?
I'll take care of it now.
I assume that the include and libiberty changes are synced
automatically from GCC to binutils - if this is not correct, please
tell me and I'll commit there as well (I'll check also in 24 hours
myself),
Andreas
--
Andreas Jaeger, aj@suse.de, http://www.suse.de/~aj
SUSE Linux Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126
[-- Attachment #2: Type: application/pgp-signature, Size: 188 bytes --]
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-28 12:43 ` Andreas Jaeger
@ 2005-05-28 21:48 ` Eli Zaretskii
2005-05-30 17:50 ` DJ Delorie
1 sibling, 0 replies; 11+ messages in thread
From: Eli Zaretskii @ 2005-05-28 21:48 UTC (permalink / raw)
To: Andreas Jaeger; +Cc: gcc-patches, binutils
> From: Andreas Jaeger <aj@suse.de>
> Cc: gcc-patches@gcc.gnu.org, binutils@sourceware.org
> Date: Sat, 28 May 2005 13:38:11 +0200
>
> > 1-week ping. Would someone with write access to GCC and Binutils CVS
> > trees please commit these changes, and also regenerate config.in and
> > configured where the ChangeLog entries say so?
>
> I'll take care of it now.
Thanks.
> I assume that the include and libiberty changes are synced
> automatically from GCC to binutils - if this is not correct, please
> tell me
Sorry, I don't know if this is correct or not.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [RFA] Eliminate warnings about snprintf declaration
2005-05-28 12:43 ` Andreas Jaeger
2005-05-28 21:48 ` Eli Zaretskii
@ 2005-05-30 17:50 ` DJ Delorie
1 sibling, 0 replies; 11+ messages in thread
From: DJ Delorie @ 2005-05-30 17:50 UTC (permalink / raw)
To: aj; +Cc: eliz, gcc-patches, binutils
> I assume that the include and libiberty changes are synced
> automatically from GCC to binutils
Yes.
^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2005-05-30 16:51 UTC | newest]
Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <41994B9D.9080809@gnu.org>
[not found] ` <01c55702$Blat.v2.4$d4764900@zahav.net.il>
[not found] ` <20050512150804.GA1808@nevyn.them.org>
[not found] ` <01c55708$Blat.v2.4$cfc9f040@zahav.net.il>
[not found] ` <20050512154716.GA3513@nevyn.them.org>
[not found] ` <01c5570e$Blat.v2.4$1c533160@zahav.net.il>
[not found] ` <20050512162453.GA5180@nevyn.them.org>
[not found] ` <01c55732$Blat.v2.4$e9bd3640@zahav.net.il>
[not found] ` <20050512205103.GB13519@nevyn.them.org>
[not found] ` <01c5586c$Blat.v2.4$a7dfc720@zahav.net.il>
[not found] ` <20050514141204.GA10684@nevyn.them.org>
2005-05-14 19:02 ` [RFA] Eliminate warnings about snprintf declaration Eli Zaretskii
2005-05-15 16:52 ` Ian Lance Taylor
2005-05-15 17:28 ` Daniel Jacobowitz
2005-05-15 19:48 ` Eli Zaretskii
2005-05-15 20:26 ` Daniel Jacobowitz
2005-05-20 12:55 ` Eli Zaretskii
2005-05-28 11:38 ` Eli Zaretskii
2005-05-28 12:43 ` Andreas Jaeger
2005-05-28 21:48 ` Eli Zaretskii
2005-05-30 17:50 ` DJ Delorie
2005-05-15 19:55 ` Eli Zaretskii
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).