]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
(quotearg_n_options): Change code to be suboptimal, in
authorJim Meyering <jim@meyering.net>
Wed, 31 Aug 2005 07:51:04 +0000 (07:51 +0000)
committerJim Meyering <jim@meyering.net>
Wed, 31 Aug 2005 07:51:04 +0000 (07:51 +0000)
order to avoid an unsuppressible warning from gcc on 64-bit systems.

lib/quotearg.c

index 13fcf0b516a65cbe880b3b51e4c2c886ced598f9..a66f882b0bc807a2e81aa9c68ec1a485d14173e5 100644 (file)
@@ -1,6 +1,6 @@
 /* quotearg.c - quote arguments for output
 
-   Copyright (C) 1998, 1999, 2000, 2001, 2002, 2004 Free Software
+   Copyright (C) 1998, 1999, 2000, 2001, 2002, 2004, 2005 Free Software
    Foundation, Inc.
 
    This program is free software; you can redistribute it and/or modify
@@ -581,7 +581,12 @@ quotearg_n_options (int n, char const *arg, size_t argsize,
 
   if (nslots <= n0)
     {
-      unsigned int n1 = n0 + 1;
+      /* FIXME: technically, the type of n1 should be `unsigned int',
+        but that evokes an unsuppressible warning from gcc-4.0.1 and
+        older.  If gcc ever provides an option to suppress that warning,
+        revert to the original type, so that the test in xalloc_oversized
+        is once again performed only at compile time.  */
+      size_t n1 = n0 + 1;
 
       if (xalloc_oversized (n1, sizeof *slotvec))
        xalloc_die ();