Make previous patch more robust.

* src/output.c (ARRAY_CARDINALITY): New macro, copied from
argmatch.h.
(output_skeleton): Use it.
Suggested by Akim Demaille.

Signed-off-by: Eric Blake <ebb9@byu.net>

diff --git a/ChangeLog b/ChangeLog
index 9b5fa94e80080803631e2f77a2b23b0909d6344a..9bf7137d18dbf2aae0ea6c235e6cdd62d9f656d1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2009-08-20  Eric Blake  <ebb9@byu.net>
 
 2009-08-20  Eric Blake  <ebb9@byu.net>
 

+       Make previous patch more robust.
+       * src/output.c (ARRAY_CARDINALITY): New macro, copied from
+       argmatch.h.
+       (output_skeleton): Use it.
+       Suggested by Akim Demaille.
+

        Import latest m4/m4.m4.
        * m4/m4.m4: Copy from autoconf 2.64.
        * configure.ac (M4_GNU_OPTION): New define.
        Import latest m4/m4.m4.
        * m4/m4.m4: Copy from autoconf 2.64.
        * configure.ac (M4_GNU_OPTION): New define.


@@ -49,12 +55,12 @@
        that perfectly supported Bison mo files.  Lower the requirement.
        http://lists.gnu.org/archive/html/bug-bison/2009-08/msg00006.html
 
        that perfectly supported Bison mo files.  Lower the requirement.
        http://lists.gnu.org/archive/html/bug-bison/2009-08/msg00006.html
 

-        * configure.ac: Require need-ngettext instead of
-        need-formatstring-macros.
-        Reported by Martin Jabocs.
-        Suggested by Bruno Haible.
-        * INSTALL: Restructure.
-        (Internationalization): New.
+       * configure.ac: Require need-ngettext instead of
+       need-formatstring-macros.
+       Reported by Martin Jabocs.
+       Suggested by Bruno Haible.
+       * INSTALL: Restructure.
+       (Internationalization): New.

 
 2009-08-14  Joel E. Denny  <jdenny@clemson.edu>
 
 
 2009-08-14  Joel E. Denny  <jdenny@clemson.edu>
 


@@ -69,7 +75,7 @@
 
        distcheck: fix.
 
 
        distcheck: fix.
 

-        * examples/calc++/Makefile.am: (EXTRA_DIST): Ship calc.stamp.
+       * examples/calc++/Makefile.am: (EXTRA_DIST): Ship calc.stamp.

 
 2009-08-06  Joel E. Denny  <jdenny@clemson.edu>
 
 
 2009-08-06  Joel E. Denny  <jdenny@clemson.edu>
 


@@ -168,13 +174,13 @@
 2009-06-10  Akim Demaille  <demaille@gostai.com>
 
        gnulib: update.
 2009-06-10  Akim Demaille  <demaille@gostai.com>
 
        gnulib: update.

-        * gnulib: Update to latest.
-        * lib/.cvsignore, lib/.gitignore, m4/.cvsignore,
-        * m4/.gitignore: Regen.
-        * src/symtab.c (symbol_from_uniqstr, semantic_type_from_uniqstr):
-        Call xalloc_die on hash_insert failures.
-        Requested by the new __warn_unused_result__ attribute of
-        hash_insert.
+       * gnulib: Update to latest.
+       * lib/.cvsignore, lib/.gitignore, m4/.cvsignore,
+       * m4/.gitignore: Regen.
+       * src/symtab.c (symbol_from_uniqstr, semantic_type_from_uniqstr):
+       Call xalloc_die on hash_insert failures.
+       Requested by the new __warn_unused_result__ attribute of
+       hash_insert.

 
 2009-05-22  Joel E. Denny  <jdenny@ces.clemson.edu>
 
 
 2009-05-22  Joel E. Denny  <jdenny@ces.clemson.edu>
 

diff --git a/src/output.c b/src/output.c
index 3fbeb302015ef4b8abf9e5533c49f07ebd5d6411..759848f7c220de109158f0bcbc4a309f4343d809 100644 (file)
--- a/src/output.c
+++ b/src/output.c
@@ -21,6 +21,7 @@
 #include <config.h>
 #include "system.h"
 
 #include <config.h>
 #include "system.h"
 

+#include <assert.h>

 #include <configmake.h>
 #include <error.h>
 #include <get-errno.h>
 #include <configmake.h>
 #include <error.h>
 #include <get-errno.h>


@@ -40,6 +41,7 @@
 #include "symtab.h"
 #include "tables.h"
 
 #include "symtab.h"
 #include "tables.h"
 

+# define ARRAY_CARDINALITY(Array) (sizeof (Array) / sizeof *(Array))

 
 static struct obstack format_obstack;
 
 
 static struct obstack format_obstack;
 


@@ -546,6 +548,7 @@ output_skeleton (void)
     argv[i++] = full_m4bison;
     argv[i++] = full_skeleton;
     argv[i++] = NULL;
     argv[i++] = full_m4bison;
     argv[i++] = full_skeleton;
     argv[i++] = NULL;

+    assert (i <= ARRAY_CARDINALITY (argv));

   }
 
   init_subpipe ();
   }
 
   init_subpipe ();