diff options
author | Eric Blake <ebb9@byu.net> | 2009-08-20 16:54:07 -0600 |
---|---|---|
committer | Eric Blake <ebb9@byu.net> | 2009-08-20 16:54:07 -0600 |
commit | b0778bdd0c2c7d5136f46ffe5c00b4c9153d5169 (patch) | |
tree | 80cdddf368988cbc73262be1316f34d0ef66f90f | |
parent | d8911864c25f5521b28fcf9e17e88436c17c59bd (diff) | |
download | bison-b0778bdd0c2c7d5136f46ffe5c00b4c9153d5169.tar.gz |
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>
(cherry picked from commit 1266b636740f0c6719d8cc11a5d569084fa37009)
-rw-r--r-- | ChangeLog | 34 | ||||
-rw-r--r-- | src/output.c | 3 |
2 files changed, 23 insertions, 14 deletions
@@ -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. * submodules/autoconf: Update to autoconf 2.64. * configure.ac (M4_GNU_OPTION): New define. @@ -121,12 +127,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> @@ -223,7 +229,7 @@ distcheck: fix. - * examples/calc++/Makefile.am: (EXTRA_DIST): Ship calc.stamp. + * examples/calc++/Makefile.am: (EXTRA_DIST): Ship calc.stamp. 2009-08-10 Joel E. Denny <jdenny@clemson.edu> @@ -553,13 +559,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-06-10 Akim Demaille <demaille@gostai.com> diff --git a/src/output.c b/src/output.c index f739f0e9..23cd8ffa 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; @@ -635,6 +637,7 @@ output_skeleton (void) argv[i++] = full_m4bison; argv[i++] = full_skeleton; argv[i++] = NULL; + assert (i <= ARRAY_CARDINALITY (argv)); } init_subpipe (); |