bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

more cleanups


From: Eric Blake
Subject: more cleanups
Date: Thu, 19 Nov 2009 16:31:20 +0000 (UTC)
User-agent: Loom/3.14 (http://gmane.org/)

I noticed these while trying to build m4 and diffutils:


From: Eric Blake <address@hidden>
Date: Thu, 19 Nov 2009 08:51:13 -0700
Subject: [PATCH 1/2] text-xvasprintf: silence compiler warnings

* tests/test-xvasprintf.c (test_xvasprintf, test_xasprintf): Mask
empty string from gcc.

Signed-off-by: Eric Blake <address@hidden>
---
 ChangeLog               |    6 ++++++
 tests/test-xvasprintf.c |   24 ++++++++++++++++--------
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 93ec669..9a868f5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-11-19  Eric Blake  <address@hidden>
+
+       text-xvasprintf: silence compiler warnings
+       * tests/test-xvasprintf.c (test_xvasprintf, test_xasprintf): Mask
+       empty string from gcc.
+
 2009-11-17  Eric Blake  <address@hidden>

        stdlib-safer: preserve cloexec flag for mkostemp[s]
diff --git a/tests/test-xvasprintf.c b/tests/test-xvasprintf.c
index 229e3fd..cc70709 100644
--- a/tests/test-xvasprintf.c
+++ b/tests/test-xvasprintf.c
@@ -65,10 +65,14 @@ test_xvasprintf (void)
       free (result);
     }

-  result = my_xasprintf ("");
-  ASSERT (result != NULL);
-  ASSERT (strcmp (result, "") == 0);
-  free (result);
+  {
+    /* Silence gcc warning about zero-length format string.  */
+    char *empty = "";
+    result = my_xasprintf (empty);
+    ASSERT (result != NULL);
+    ASSERT (strcmp (result, "") == 0);
+    free (result);
+  }

   result = my_xasprintf ("%s", "foo");
   ASSERT (result != NULL);
@@ -100,10 +104,14 @@ test_xasprintf ()
       free (result);
     }

-  result = xasprintf ("");
-  ASSERT (result != NULL);
-  ASSERT (strcmp (result, "") == 0);
-  free (result);
+  {
+    /* Silence gcc warning about zero-length format string.  */
+    char *empty = "";
+    result = xasprintf (empty);
+    ASSERT (result != NULL);
+    ASSERT (strcmp (result, "") == 0);
+    free (result);
+  }

   result = xasprintf ("%s", "foo");
   ASSERT (result != NULL);
-- 
1.6.4.2


>From e1659df56e80448785886a593583d35f91a75d40 Mon Sep 17 00:00:00 2001
From: Eric Blake <address@hidden>
Date: Thu, 19 Nov 2009 09:12:34 -0700
Subject: [PATCH 2/2] xalloc-die-tests: add missing library

* modules/xalloc-die-tests (Makefile.am): Add LDADD line.

Signed-off-by: Eric Blake <address@hidden>
---
 ChangeLog                |    3 +++
 modules/xalloc-die-tests |    1 +
 2 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 9a868f5..ed9e8f5 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2009-11-19  Eric Blake  <address@hidden>

+       xalloc-die-tests: add missing library
+       * modules/xalloc-die-tests (Makefile.am): Add LDADD line.
+
        text-xvasprintf: silence compiler warnings
        * tests/test-xvasprintf.c (test_xvasprintf, test_xasprintf): Mask
        empty string from gcc.
diff --git a/modules/xalloc-die-tests b/modules/xalloc-die-tests
index ed6a470..9b430cb 100644
--- a/modules/xalloc-die-tests
+++ b/modules/xalloc-die-tests
@@ -8,3 +8,4 @@ Makefile.am:
 TESTS += test-xalloc-die.sh
 TESTS_ENVIRONMENT += EXEEXT='@EXEEXT@'
 check_PROGRAMS += test-xalloc-die
+test_xalloc_die_LDADD = $(LDADD) @LIBINTL@
-- 
1.6.4.2







reply via email to

[Prev in Thread] Current Thread [Next in Thread]