test-skeleton.c: xmalloc, xcalloc, xrealloc are potentially unused

__attribute__ ((used)) means that the function has to be
emitted in assembly because it is referenced in ways the
compiler cannot detect (such as asm statements, or some
post-processing on the generated assembly).

The unused attribute needs to come first, otherwise it is
applied to the return type and not the function definition.

2 files changed, 8 insertions, 3 deletions

diff --git a/ChangeLog b/ChangeLog
index 98d78c9c34..5f9edb42eb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2016-06-23  Florian Weimer  <fweimer@redhat.com>
+
+	* test-skeleton.c (xmalloc, xcalloc, xrealloc): Mark as
+	potentially unused.
+
 2016-06-22  Florian Weimer  <fweimer@redhat.com>
 
 	* test-skeleton.c (write_message): New function.
diff --git a/test-skeleton.c b/test-skeleton.c
index 49808b3d85..0be4af1daf 100644
--- a/test-skeleton.c
+++ b/test-skeleton.c
@@ -78,8 +78,8 @@ oom_error (const char *fn, size_t size)
 }
 
 /* Allocate N bytes of memory dynamically, with error checking.  */
+__attribute__ ((unused))
 static void *
-__attribute__ ((used))
 xmalloc (size_t n)
 {
   void *p;
@@ -91,8 +91,8 @@ xmalloc (size_t n)
 }
 
 /* Allocate memory for N elements of S bytes, with error checking.  */
+__attribute__ ((unused))
 static void *
-__attribute__ ((used))
 xcalloc (size_t n, size_t s)
 {
   void *p;
@@ -105,8 +105,8 @@ xcalloc (size_t n, size_t s)
 
 /* Change the size of an allocated block of memory P to N bytes,
    with error checking.  */
+__attribute__ ((unused))
 static void *
-__attribute__ ((used))
 xrealloc (void *p, size_t n)
 {
   p = realloc (p, n);