+2012-01-06 Jim Meyering <meyering@redhat.com>
+
+ build: avoid warnings about set-but-not-used variables
+ * lib/bitset_stats.c (bitset_stats_list): Remove declaration
+ and set of unused local, "type".
+ * lib/ebitset.c (ebitset_init): Likewise, for "size".
+ * lib/lbitset.c (lbitset_not): Likewise, for "elt".
+
+2011-08-21 Joel E. Denny <joeldenny@joeldenny.org>
+
+ lalr1.cc: remove trailing comma from enumerator list.
+ * data/lalr1.cc (yy::parser): Here. This suppresses a -pedantic
+ warning.
+
+2011-08-21 Joel E. Denny <joeldenny@joeldenny.org>
+
+ tests: add -pedantic for --enable-gcc-warnings.
+ This should help to avoid some portability problems. For example,
+ it would have revealed the empty unions fixed by the last patch
+ * configure.ac (WARN_CFLAGS_TESTS, WARN_CXXFLAGS_TEST): Implement.
+ * tests/synclines.at (AT_TEST_SYNCLINE): Avoid -pedantic warning
+ about an empty translation unit.
+
+2011-08-21 Joel E. Denny <joeldenny@joeldenny.org>
+
+ tests: fix empty unions.
+ Empty unions are not accepted by the ISO C99 grammar or by at
+ least some versions of Sun Studio. Reported by Wolfgang S. Kechel
+ at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-08/msg00003.html>.
+ * NEWS (2.5.1): Document fix.
+ * THANKS (Wolfgang S. Kechel): Add.
+ * tests/existing.at (GNU Cim Grammar)
+ (GNU pic (Groff 1.18.1) Grammar.): Remove empty %union.
+
+2011-07-24 Joel E. Denny <joeldenny@joeldenny.org>
+
+ global: remove unnecessary horizontal tabs.
+ This change was made by applying emacs' untabify function to
+ nearly all files in Bison's repository. Required tabs in make
+ files, ChangeLog, regexps, and test code were manually skipped.
+ Other notable exceptions and changes are listed below.
+ * bootstrap: Skip because we sync this with gnulib.
+ * data/m4sugar/foreach.m4
+ * data/m4sugar/m4sugar.m4: Skip because we sync these with
+ Autoconf.
+ * djgpp: Skip because I don't know how to test djgpp properly, and
+ this code appears to be unmaintained anyway.
+ * README-hacking (Hacking): Specify that tabs should be avoided
+ where not required.
+
+2011-07-10 Joel E. Denny <joeldenny@joeldenny.org>
+
+ build: avoid YACC typo inherited from Autoconf.
+ The typo shows up in the `configure --help' output. Reported by
+ W.C.A. Wijngaards and Paul Eggert at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-07/msg00000.html>.
+ * README-hacking (Release Procedure): Remind about updating
+ maintainer tools before a release.
+ * THANKS (W.C.A. Wijngaards): Add.
+ * configure.ac (AC_PREREQ): Set to 2.68, which fixes the YACC
+ typo.
+
+2011-07-10 Joel E. Denny <joeldenny@joeldenny.org>
+
+ build: create xz instead of bzip2 tarballs.
+ Suggested by Jim Meyering at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-06/msg00012.html>.
+ * README-hacking (Release Procedure): Update example.
+ * configure.ac (AM_INIT_AUTOMAKE): Replace dist-bzip2 with
+ dist-xz.
+
+2011-05-29 Joel E. Denny <joeldenny@joeldenny.org>
+
+ doc: clean up references to `Locations Overview'.
+ * doc/bison.texinfo (Locations Overview): Rename node to...
+ (Locations): ... this, which is its section name. Update menus to
+ match that and not to use the title `Tracking Locations', which is
+ a different section.
+
+2011-05-29 Joel E. Denny <joeldenny@joeldenny.org>
+
+ doc: clean up references to `Tracking Locations'.
+ * doc/bison.texinfo (Locations): Rename node to...
+ (Tracking Locations): ... this, which is its section name, and
+ update all cross references. The trouble is that there is another
+ node about locations. Its section name is `Locations', which was
+ easily confused with the node name of this node. Moreover, its
+ node name is `Locations Overview', which was mistakenly used to
+ label some cross references to this node.
+
+2011-05-29 Joel E. Denny <joeldenny@joeldenny.org>
+
+ doc: clean up `Named References' a little.
+ * doc/bison.texinfo (Named References): Rename section to match
+ the node name, and update all cross references. Don't use the
+ acronyms RHS and LHS given that we don't expand them anywhere in
+ the manual. Try to use @samp and @code properly. Mention that
+ named references are experimental as we've already stated in NEWS.
+ (Destructor Decl): Expand the only remaining use of RHS or LHS.
+
+2011-05-29 Joel E. Denny <joeldenny@joeldenny.org>
+
+ doc: motivate named references.
+ Suggested by Hans Aberg at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-05/msg00008.html>.
+ * doc/bison.texinfo (Named References): Explain briefly how
+ they're better than the traditional positional references.
+
+2011-05-29 Joel E. Denny <joeldenny@joeldenny.org>
+
+ doc: discuss named references after locations.
+ Reported by Hans Aberg at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-05/msg00008.html>.
+ * NEWS (2.5.1): Document.
+ * doc/bison.texinfo (Named References): Because it discusses
+ locations in addition to semantic values, move this subsection out
+ of the section `Defining Language Semantics', where locations have
+ not yet been introduced, to be a new section after the following
+ section, `Tracking Locations'.
+
+2011-05-29 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Prepare for the possibility of a 2.5.1 release.
+ * NEWS (2.5.1): New heading.
+
+2011-05-14 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Version 2.5.
+ * NEWS (2.5): Set date.
+
+2011-05-14 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Don't use IF_LINT in Bison sources.
+ It creates unnecessary differences between the sources that Bison
+ maintainers build and test (given that maintainers normally
+ configure with --enable-gcc-warnings) and the sources that Bison
+ users build. Instead, use PACIFY_CC, which doesn't. This change
+ fixes compiler warnings reported by Tys Lefering at
+ <http://lists.gnu.org/archive/html/bison-patches/2011-05/msg00004.html>.
+ * configure.ac: Don't AC_DEFINE lint regardless of the configure
+ options. This change affects imported gnulib sources, where
+ IF_LINT still appears and depends on lint.
+ * src/scan-gram.l, src/scan-skel.l: Replace uses of IF_LINT with
+ PACIFY_CC.
+ * src/system.h (IF_LINT): Remove cpp macro.
+ (PACIFY_CC): New cpp macro.
+
+2011-05-01 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Fix precedence for end token.
+ Since Bison 2.3b, which restored the ability of precedence
+ directives to assign user token numbers, doing so for user token
+ number 0 has produced an assertion failure.
+ * NEWS (2.5): Document fix.
+ * src/symtab.c (symbol_user_token_number_set): In the case of the
+ end token, don't decrement ntokens if it was never incremented.
+ * tests/regression.at (Token number in precedence declaration):
+ Extend.
+
+2011-05-01 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Pacify -DGNULIB_POSIXCHECK.
+ * bootstrap.conf (gnulib_modules): Add all modules suggested by
+ -DGNULIB_POSIXCHECK.
+ * src/files.c (file_name_split)
+ * src/getargs.c (getargs)
+ * src/location.c (boundary_set_from_string)
+ * src/output.c (output_skeleton)
+ * src/parse-gram.y (prologue_declaration)
+ * src/scan-gram.l (handle_syncline)
+ * src/symtab.c (symbol_new): Use mbschr and mbsrchr instead of
+ strchr and strrchr. In the cases of command-line options, file
+ names, and thus locations, functionality may be improved. In the
+ case of symbol names, there should be no functional difference as
+ all characters are ASCII, so the intended benefit is just warning
+ suppression.
+
+2011-05-01 Joel E. Denny <joeldenny@joeldenny.org>
+
+ * NEWS (2.5): Fix minor typos.
+
+2011-05-01 Joel E. Denny <joeldenny@joeldenny.org>
+
+ doc: clean up quotation style in NEWS.
+ * NEWS (2.5): For multi-character tokens in example grammar rules,
+ use "token" not 'token'. In English, use a consistent quotation
+ style; we might as well follow the precedent seen in info pages:
+ `quote' not "quote".
+
+2011-04-16 Joel E. Denny <joeldenny@joeldenny.org>
+
+ tests: pacify gcc 4.6.0's -Wunused-but-set-variable.
+ Reported by Jim Meyering at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-04/msg00002.html>.
+ * tests/actions.at
+ (Default %printer and %destructor for mid-rule values): Define
+ YYLLOC_DEFAULT so that it uses its Rhs argument.
+
+2011-04-16 Joel E. Denny <joeldenny@joeldenny.org>
+
+ glr.c: omit yyresolveLocations when locations are disabled.
+ This prevents gcc 4.6.0's -Wunused-but-set-variable from warning
+ about yyresolveLocations's local yyrhsloc. When locations are
+ enabled, there's no such warning because YYLLOC_DEFAULT then uses
+ yyrhsloc. Reported by Jim Meyering at
+ <http://lists.gnu.org/archive/html/bug-bison/2011-04/msg00002.html>.
+ * data/glr.c (yyresolveLocations): Omit definition when locations
+ are disabled.
+ (yyresolveValue): Omit yyresolveLocations invocation when
+ locations are disabled.
+
+2011-04-16 Joel E. Denny <joeldenny@joeldenny.org>
+
+ gnulib, autoconf: update.
+ * README-hacking (Updating a submodule): Give advice on how to
+ determine the versions of gnulib and autoconf to which we should
+ update.
+ (Release Procedure): Note that submodules should be updated.
+ * bootstrap.conf (gnulib_modules): Rename pipe to spawn-pipe as
+ suggested in updated gnulib NEWS.
+ * gnulib: Choose a stable snapshot according to advice in Bison's
+ README-hacking.
+ * po/POTFILES.in (lib/pipe.c): Rename to...
+ (lib/spawn-pipe.c): ... this.
+ * src/output.c: Update to include spawn-pipe.h.
+ * submodules/autoconf: Update to latest for improvement in m4.m4
+ that excludes M4 with buggy strstr. The only other changes to
+ files that we use are copyright updates.
+
+2011-04-09 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Fix missing updates to GPLv3.
+ Reported by Tys Lefering at
+ <http://lists.gnu.org/archive/html/bison-patches/2011-04/msg00000.html>.
+ * src/print-xml.c, src/print-xml.h: In these files.
+
+2011-03-27 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Add -Wconflicts-sr and -Wconflicts-rr.
+ Thus, conflict reports are now affected by -Werror and -Wnone
+ (unless %expect or %expect-rr is specified). Reported by George
+ Neuner at
+ <http://lists.gnu.org/archive/html/bug-bison/2010-08/msg00002.html>.
+ * NEWS (2.5): Document.
+ * doc/bison.texinfo (Bison Options): Document.
+ * src/complain.c, src/complain.h (set_warning_issued): Export
+ function.
+ * src/conflicts.c (conflicts_print): Suppress conflict report
+ based on -Wno-conflicts-sr and -Wno-conflicts-rr, and treat
+ conflicts as errors if -Werror.
+ * src/getargs.c (warnings_flag): Initialize with
+ warnings_conflicts_sr and warnings_conflicts_rr as well.
+ (warnings_args, warnings_types): Add entries for
+ warnings_conflicts_sr and warnings_conflicts_rr.
+ (usage): Update.
+ * src/getargs.h (enum warnings): Add entries for
+ warnings_conflicts_sr and warnings_conflicts_rr.
+ * tests/conflicts.at (-W versus %expect and %expect-rr): New test
+ group.
+ * tests/local.at (AT_BISON_CHECK_NO_XML): Update now that the
+ conflict report can produce a "warnings being treated as errors"
+ message. Also, check that stderr is now fully scrubbed by -Wnone
+ when the exit status is 0.
+
+2011-03-27 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Pacify maintainer-check-posix.
+ Adding command-line options after the grammar file name is not
+ permitted, so disable checks that do that when
+ maintainer-check-posix is running.
+ * tests/local.at (AT_BISON_CHECK_NO_XML): Don't run the
+ problematic checks when POSIXLY_CORRECT=1. Also, for readability,
+ remove an unnecessary m4_if.
+
+2011-03-27 Joel E. Denny <joeldenny@joeldenny.org>
+
+ Add -Wother so -Wnone suppresses all warnings.
+ Reported by George Neuner at
+ <http://lists.gnu.org/archive/html/bug-bison/2010-08/msg00002.html>.
+ * NEWS (2.5): Document.
+ * THANKS (George Neuner): Add.
+ * doc/bison.texinfo (Bison Options): Document.
+ * src/complain.c, src/complain.h
+ (warn_at, warn_at_indent, warn): Suppress warning if -Wno-other.
+ (midrule_value_at): New warning function, similar to yacc_at in
+ that it's controlled by its own warning category.
+ * src/getargs.c (warnings_flag): Initialize to warnings_other.
+ (warnings_args, warnings_types): Add entry for warnings_other.
+ (usage): Update.
+ * src/getargs.h (enum warnings): Add entry for warnings_other.
+ * src/gram.c (grammar_rules_useless_report): If -Wno-other, then
+ don't print useless rules.
+ * src/reader.c (symbol_should_be_used): Rather than adjusting the
+ return value based on whether midrule value warnings are enabled,
+ accept a new parameter for telling the caller whether true is
+ being returned for a potential midrule warning.
+ (grammar_rule_check): Use midrule_value_at for midrule value
+ warnings, and continue to use warn_at for all other warnings. Let
+ them check whether the warnings are enabled.
+ * tests/local.at (AT_BISON_CHECK): Update documentation.
+ (AT_BISON_CHECK_NO_XML): Check that -Wnone and --warnings=none
+ disable all warnings exercised in the test suite.
+
2011-03-27 Joel E. Denny <joeldenny@joeldenny.org>
Don't let -Wnone disable -Werror.