From: Jim Meyering <meyering@redhat.com>
Date: Wed, 13 Oct 2010 08:02:17 +0000 (+0200)
Subject: test-inttostr: avoid shadowing warnings
X-Git-Tag: v0.1~3690
X-Git-Url: http://erislabs.net/gitweb/?a=commitdiff_plain;h=7160b4752c2dadb04f6f365204f8aa84fa89eef7;p=gnulib.git

test-inttostr: avoid shadowing warnings

* tests/test-inttostr.c (main): Rename local, "buf" to "b",
and use malloc rather than the stack for the same reason as
mentioned in the comment justifying the other allocation.
---

diff --git a/ChangeLog b/ChangeLog
index 25790a619..3f0ff01be 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-10-13  Jim Meyering  <meyering@redhat.com>
+
+	test-inttostr: avoid shadowing warnings
+	* tests/test-inttostr.c (main): Rename local, "buf" to "b",
+	and use malloc rather than the stack for the same reason as
+	mentioned in the comment justifying the other allocation.
+
 2010-10-11  Bruno Haible  <bruno@clisp.org>
 
 	stdlib: Allow multiple gnulib generated replacements to coexist.
diff --git a/tests/test-inttostr.c b/tests/test-inttostr.c
index bf18621e4..1ff35c08c 100644
--- a/tests/test-inttostr.c
+++ b/tests/test-inttostr.c
@@ -65,13 +65,15 @@
 int
 main (void)
 {
-  char buf[2];
+  size_t b_size = 2;
+  char *b = malloc (b_size);
+  ASSERT (b);
 
   /* Ideally we would rely on the snprintf-posix module, in which case
      this guard would not be required, but due to limitations in gnulib's
      implementation (see modules/snprintf-posix), we cannot.  */
-  if (snprintf (buf, sizeof buf, "%ju", (uintmax_t) 3) == 1
-      && buf[0] == '3' && buf[1] == '\0')
+  if (snprintf (b, b_size, "%ju", (uintmax_t) 3) == 1
+      && b[0] == '3' && b[1] == '\0')
     {
       CK (int,          inttostr);
       CK (unsigned int, uinttostr);