From 1266b636740f0c6719d8cc11a5d569084fa37009 Mon Sep 17 00:00:00 2001
From: Eric Blake <ebb9@byu.net>
Date: Thu, 20 Aug 2009 16:43:07 -0600
Subject: [PATCH] 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>
---
 ChangeLog    | 34 ++++++++++++++++++++--------------
 src/output.c |  3 +++
 2 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 9b5fa94e..9bf7137d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 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.
@@ -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
 
-        * 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>
 
@@ -69,7 +75,7 @@
 
 	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>
 
@@ -168,13 +174,13 @@
 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>
 
diff --git a/src/output.c b/src/output.c
index 3fbeb302..759848f7 100644
--- a/src/output.c
+++ b/src/output.c
@@ -21,6 +21,7 @@
 #include <config.h>
 #include "system.h"
 
+#include <assert.h>
 #include <configmake.h>
 #include <error.h>
 #include <get-errno.h>
@@ -40,6 +41,7 @@
 #include "symtab.h"
 #include "tables.h"
 
+# define ARRAY_CARDINALITY(Array) (sizeof (Array) / sizeof *(Array))
 
 static struct obstack format_obstack;
 
@@ -546,6 +548,7 @@ output_skeleton (void)
     argv[i++] = full_m4bison;
     argv[i++] = full_skeleton;
     argv[i++] = NULL;
+    assert (i <= ARRAY_CARDINALITY (argv));
   }
 
   init_subpipe ();
-- 
2.45.2