X-Git-Url: https://git.saurik.com/bison.git/blobdiff_plain/519d000408fc583144fb9c7a18d3718cefaf0de7..89eb3c76532c0c5f9f3207474c55c300a239fce8:/ChangeLog

diff --git a/ChangeLog b/ChangeLog
index 0b026dfe..54906055 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,91 @@
+2006-10-31  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+	Finish implementing --warnings=error, which should not be implied by
+	--warnings=all (or by its synonyms -W and --warnings without
+	subarguments).
+	* src/complain.c (set_warning_issued): New function to report that
+	warnings are being treated as errors and to record an error if so.
+	Invoke...
+	(warn_at, warn): ... here.
+	* src/getargs.c (warnings_args, warnings_types): Reorder so that
+	"error - warnings are errors" does not appear above "all - all of the
+	above".
+	(getargs): For -W and --warnings without subarguments, don't let
+	FLAGS_ARGMATCH set warnings_error in warnings_flag.
+	* src/getargs.h (enum warnings): Unset warnings_error in warnings_all.
+
+2006-10-31  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+	* src/getargs.c (flags_argmatch): Don't cause segmentation fault for
+	empty subargument list.  For example: `bison  --warnings=  parser.y'.
+
+2006-10-21  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+	* data/push.c, data/yacc.c: Make sure there's a newline at the end of
+	the parser header file so that gcc doesn't warn.
+
+2006-10-21  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+	Split the default %destructor/%printer into two kinds: <*> and <!>.
+	Discussed starting at
+	<http://lists.gnu.org/archive/html/bison-patches/2006-09/msg00060.html>.
+	* NEWS (2.3a+): Mention.
+	* doc/bison.texinfo (Freeing Discarded Symbols): Document this and the
+	previous change today related to mid-rules.
+	(Bison Symbols): Remove %symbol-default and add <*> and <!>.
+	* src/parser-gram.y (PERCENT_SYMBOL_DEFAULT): Remove.
+	(TYPE_TAG_ANY): Add as <*>.
+	(TYPE_TAG_NONE): Add as <!>.
+	(generic_symlist_item): Remove RHS for %symbol-default and add RHS's
+	for <*> and <!>.
+	* src/scan-gram.l (PERCENT_SYMBOL_DEFAULT): Remove.
+	(TYPE_TAG_ANY, TYPE_TAG_NONE): Add.
+	* src/symlist.c (symbol_list_default_new): Split into tagged and
+	tagless versions.
+	(symbol_list_destructor_set, symbol_list_printer_set): Split
+	SYMLIST_DEFAULT case into SYMLIST_DEFAULT_TAGGED and
+	SYMLIST_DEFAULT_TAGLESS.
+	* src/symlist.h: Update symbol_list_default*_new prototypes.
+	(symbol_list.content_type): Split enum value SYMLIST_DEFAULT into
+	SYMLIST_DEFAULT_TAGGED and SYMLIST_DEFAULT_TAGLESS.
+	* src/symtab.c (default_destructor, default_destructor_location,
+	default_printer, default_printer_location): Split each into tagged and
+	tagless versions.
+	(symbol_destructor_get, symbol_destructor_location_get,
+	symbol_printer_get, symbol_printer_location_get): Implement tagged
+	default and tagless default cases.
+	(default_destructor_set, default_printer_set): Split each into tagged
+	and tagless versions.
+	* src/symtab.h: Update prototypes.
+	* tests/actions.at (Default %printer and %destructor): Rename to...
+	(Default tagless %printer and %destructor): ... this, and extend.
+	(Per-type %printer and %destructor): Rename to...
+	(Default tagged and per-type %printer and %destructor): ... this, and
+	extend.
+	(Default %printer and %destructor for user-defined end token): Extend.
+	(Default %printer and %destructor are not for error or $undefined):
+	Update.
+	(Default %printer and %destructor are not for $accept): Update.
+	(Default %printer and %destructor for mid-rule values): Extend.
+	* tests/input.at (Default %printer and %destructor redeclared): Extend.
+	(Unused values with default %destructor): Extend.
+
+2006-10-21  Joel E. Denny  <jdenny@ces.clemson.edu>
+
+	Don't apply the default %destructor/%printer to an unreferenced midrule
+	value.  Mentioned at
+	<http://lists.gnu.org/archive/html/bison-patches/2006-09/msg00104.html>.
+	* src/symtab.c (dummy_symbol_get): Name all dummy symbols initially
+	like $@n instead of just @n so that the default %destructor/%printer
+	logic doesn't see them as user-defined symbols.
+	(symbol_is_dummy): Check for both forms of the name.
+	* src/reader.c (packgram): Remove the `$' from each midrule symbol
+	name for which the midrule value is referenced in any action.
+	* tests/actions.at (Default %printer and %destructor for mid-rule
+	values): New test.
+	* tests/regression.at (Rule Line Numbers, Web2c Report): Update output
+	for change to dummy symbol names.
+
 2006-10-20  Joel E. Denny  <jdenny@ces.clemson.edu>
 
 	Warn about unset midrule $$ if the corresponding $n is used.