Akim Demaille [Fri, 21 Dec 2012 15:52:11 +0000 (16:52 +0100)]
maint: more syntax-checks
* cfg.mk (sc_prohibit_tab_based_indentation, sc_prohibit_test_backticks)
(sc_preprocessor_indentation, sc_space_before_open_paren): New,
stolen from Coreutils (
2e9f5ca4ebbbdb6a9fa2dd3d5add3f7720a172d7).
Akim Demaille [Fri, 21 Dec 2012 15:51:17 +0000 (16:51 +0100)]
debug: no longer generate tabs
* src/closure.c, src/derives.c, src/nullable.c, tests/sets.at: Use
spaces.
Akim Demaille [Fri, 21 Dec 2012 15:45:30 +0000 (16:45 +0100)]
style changes: run cppi
Run it in src/ for a start.
* src/AnnotationList.h, src/InadequacyList.h, src/Sbitset.h,
* src/closure.c, src/complain.h, src/flex-scanner.h, src/getargs.h,
* src/gram.h, src/graphviz.h, src/ielr.h, src/location.h,
* src/muscle-tab.h, src/named-ref.h, src/relation.h, src/scan-code.h,
* src/state.h, src/symtab.h, src/system.h, src/uniqstr.h:
Reindent preprocessor directives.
Akim Demaille [Fri, 21 Dec 2012 15:36:32 +0000 (16:36 +0100)]
style changes: untabify
* data/xslt/xml2dot.xsl, data/xslt/xml2text.xsl, m4/flex.m4,
* tests/glr-regression.at, tests/torture.at: here.
Akim Demaille [Fri, 21 Dec 2012 12:23:54 +0000 (13:23 +0100)]
tests: be robust to set -e.
* examples/test (run): here.
Akim Demaille [Wed, 19 Dec 2012 09:18:03 +0000 (10:18 +0100)]
variants: prohibit simple copies
The "variant" structure provides a means to store, in a typeless way,
C++ objects. Manipulating it without provide the type of the stored
content is doomed to failure. So provide a means to copy in a type
safe way, and prohibit typeless assignments.
* data/c++.m4 (symbol_type::move): New.
* data/lalr1.cc: Use it.
* data/variant.hh (b4_variant_define): Provide variant::copy.
Let variant::operator= abort.
We cannot undefine it, yet, as it is still uses by the implicit
assigment in symbols, which must also be disabled.
Akim Demaille [Wed, 19 Dec 2012 09:09:07 +0000 (10:09 +0100)]
variant: more assertions
Equip variants with more checking code. Provide a means to request
includes.
* data/variant.hh (b4_variant_includes): New.
* data/lalr1.cc: Use it.
* data/variant.hh (variant::built): Define at the end, as a private member.
(variant::tname): New.
Somewhat makes "built" useless, but let's keep both for a start, in
case using "typeinfo" is considered unacceptable in some environments.
Fix some formatting issues.
Akim Demaille [Wed, 19 Dec 2012 10:19:52 +0000 (11:19 +0100)]
gnulib: update
Akim Demaille [Wed, 19 Dec 2012 10:17:29 +0000 (11:17 +0100)]
skeletons: fix output directives
* data/lalr1.cc, data/location.cc, data/glr.cc: Use b4_output_begin.
Broken during a merge.
Akim Demaille [Wed, 19 Dec 2012 10:05:56 +0000 (11:05 +0100)]
yacc.c: style changes
* data/yacc.c (b4_lex_param): Provide arguments with a name.
Akim Demaille [Wed, 19 Dec 2012 10:01:52 +0000 (11:01 +0100)]
glr.cc: simplifying the handling of parse/lex params
The fact that glr.cc uses glr.c makes the handling of parse params
more complex, as the parser object of glr.cc must be passed to the
parse function of glr.c. Yet not all the functions need access to
the parser object.
* data/glr.cc (b4_parse_param_wrap): New.
Use them.
Akim Demaille [Wed, 19 Dec 2012 09:48:09 +0000 (10:48 +0100)]
glr: rename lex params
* data/glr.c (b4_lex_param): Rename as...
(b4_lex_formals): this, for consistency.
Provide arguments a name.
(LEX): Adjust.
Akim Demaille [Wed, 19 Dec 2012 09:36:40 +0000 (10:36 +0100)]
glr.c: move function declaration earlier
* data/glr.c (yypstack, yypdumpstack): Declare earlier, to make
it easier to call them from other functions.
Akim Demaille [Wed, 19 Dec 2012 09:33:58 +0000 (10:33 +0100)]
%define variables: backward compatibility
* src/muscle-tab.c (muscle_percent_variable_update): Accept lex_symbol.
Reported by Roland Levillain.
Akim Demaille [Sat, 15 Dec 2012 20:54:27 +0000 (21:54 +0100)]
diagnostics: improve -fcaret for list of accepted values
Instead of
input.y:1.9-21: error: invalid value for %define variable 'api.push-pull': 'neither'
%define api.push_pull "neither"
^^^^^^^^^^^^^
input.y:1.9-21: accepted value: 'pull'
%define api.push_pull "neither"
^^^^^^^^^^^^^
input.y:1.9-21: accepted value: 'push'
%define api.push_pull "neither"
^^^^^^^^^^^^^
input.y:1.9-21: accepted value: 'both'
%define api.push_pull "neither"
^^^^^^^^^^^^^
report
input.y:1.9-21: error: invalid value for %define variable 'api.push-pull': 'neither'
%define api.push_pull "neither"
^^^^^^^^^^^^^
input.y:1.9-21: accepted value: 'pull'
input.y:1.9-21: accepted value: 'push'
input.y:1.9-21: accepted value: 'both'
* src/complain.h (no_caret): New.
* src/complain.c (error_message): Use it.
* src/muscle-tab.c (muscle_percent_define_check_values): Use it.
* src/scan-skel.l (flag): Ditto.
* tests/input.at: Adjust and check.
Akim Demaille [Sat, 15 Dec 2012 14:27:10 +0000 (15:27 +0100)]
skeletons: simplify the handling of default api.location.type
* data/bison.m4 (b4_bison_locations_if): New.
* data/glr.cc, data/lalr1.cc: Use it.
Akim Demaille [Sat, 15 Dec 2012 16:16:28 +0000 (17:16 +0100)]
tests: address syntax-check failures
* cfg.mk: Ignore failures in timevar (uses GCC style configuration,
not gnulib's).
* doc/local.mk: Space changes.
* lib/main.c, tests/calc.at: Remove useless HAVE_ tests.
Akim Demaille [Sat, 15 Dec 2012 15:20:51 +0000 (16:20 +0100)]
remove duplicate definitions
* src/system.h: here, inherited from a merge.
Akim Demaille [Sat, 15 Dec 2012 07:19:59 +0000 (08:19 +0100)]
doc: fix dependencies
* doc/local.mk: here.
Akim Demaille [Fri, 14 Dec 2012 15:38:02 +0000 (16:38 +0100)]
doc: style fixes
* doc/bison.texi: Add a couple of missing @var and @code.
Theophile Ranquet [Tue, 4 Dec 2012 15:09:24 +0000 (16:09 +0100)]
doc: fix build dependencies
Suggested by Nick Bowler
<http://lists.gnu.org/archive/html/bug-automake/2012-12/msg00001.html>
* doc/local.mk: Avoid overwriting Automake's rules.
Akim Demaille [Fri, 14 Dec 2012 12:56:11 +0000 (13:56 +0100)]
Merge branch 'origin/maint'
* origin/maint:
maint: credit Wojciech Polak
maint: post-release administrivia
version 2.7
yacc.c: scope reduction
tests: C90 compliance
fix C90 compliance
glr.c: scope reduction
gnulib: update
Conflicts:
NEWS
gnulib
src/scan-gram.l
src/system.h
Theophile Ranquet [Tue, 11 Dec 2012 12:23:44 +0000 (13:23 +0100)]
symtab: add missing initializations
* src/symtab.c (semantic_type_new): Here.
Theophile Ranquet [Tue, 11 Dec 2012 12:16:22 +0000 (13:16 +0100)]
symtab: fix some leaks
* src/symlist.c (symbol_list_free): Deep free it.
* src/symtab.c (symbols_free, semantic_types_sorted): Free it too.
(symbols_do, sorted): Call by address.
Theophile Ranquet [Mon, 10 Dec 2012 18:28:43 +0000 (19:28 +0100)]
tests: remove use of PARSE_PARAM
* tests/header.at: Here.
Akim Demaille [Thu, 13 Dec 2012 08:25:34 +0000 (09:25 +0100)]
maint: credit Wojciech Polak
* NEWS, THANKS: He is the author of XML support (including XSLTs).
Akim Demaille [Wed, 12 Dec 2012 15:23:44 +0000 (16:23 +0100)]
maint: post-release administrivia
* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.
Akim Demaille [Wed, 12 Dec 2012 15:07:46 +0000 (16:07 +0100)]
version 2.7
* NEWS: Record release date.
Akim Demaille [Wed, 12 Dec 2012 11:38:43 +0000 (12:38 +0100)]
yacc.c: scope reduction
* data/yacc.c (yysyntax_error): here.
Akim Demaille [Wed, 12 Dec 2012 11:38:28 +0000 (12:38 +0100)]
tests: C90 compliance
* tests/synclines.at: here.
Akim Demaille [Wed, 12 Dec 2012 10:37:02 +0000 (11:37 +0100)]
fix C90 compliance
* data/glr.c, src/graphviz.h, src/ielr.c, src/scan-gram.l,
* src/system.h, tests/actions.at, tests/glr-regression.at: Do not
use // comments.
Do not introduce variables after statements.
Provide "main" with a return value.
Akim Demaille [Wed, 12 Dec 2012 10:28:24 +0000 (11:28 +0100)]
glr.c: scope reduction
* data/glr.c (yyreportSyntaxError): Reduce the scope of yysize1 (now
yysz).
Akim Demaille [Tue, 11 Dec 2012 10:39:51 +0000 (11:39 +0100)]
gnulib: update
Theophile Ranquet [Mon, 10 Dec 2012 16:01:55 +0000 (17:01 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
news: prepare for forthcoming release
doc: explain how mid-rule actions are translated
error: use better locations for unused midrule values
doc: various minor improvements and fixes
tests: ignore more useless compiler warnings
tests: be robust to C being compiled with a C++11 compiler
build: beware of Clang++ not supporting POSIXLY_CORRECT
maint: post-release administrivia
version 2.6.90
build: fix syntax-check error.
cpp: simplify the Flex version checking macro
news: improve the carets example and fix a typo
cpp: improve the Flex version checking macro
carets: improve the code
maint: update news
build: keep -Wmissing-declarations and -Wmissing-prototypes for modern GCCs
build: drop -Wcast-qual
gnulib: update
Conflicts:
NEWS
doc/Makefile.am
doc/bison.texi
gnulib
src/reader.c
tests/actions.at
tests/atlocal.in
tests/input.at
Akim Demaille [Sun, 9 Dec 2012 15:50:12 +0000 (16:50 +0100)]
news: prepare for forthcoming release
* NEWS: Fill paragraph.
Reorder.
Update examples.
Remove line for 2.6.90.
Akim Demaille [Sun, 9 Dec 2012 15:49:58 +0000 (16:49 +0100)]
doc: explain how mid-rule actions are translated
* doc/bison.texi (Actions in Mid-Rule): Mention and use named references.
Split into three subsections, among which...
(Mid-Rule Action Translation): this new section.
Akim Demaille [Sun, 9 Dec 2012 11:06:21 +0000 (12:06 +0100)]
error: use better locations for unused midrule values
On
%%
exp: {;} {$$;} { $$ = $1; }
instead of reporting (with -fcaret -Wmidrule-value)
midrule.y:2.6-8: warning: unset value: $$ [-Wmidrule-values]
exp: {;} {$$;} { $$ = $1; }
^^^
midrule.y:2.6-27: warning: unused value: $2 [-Wmidrule-values]
exp: {;} {$$;} { $$ = $1; }
^^^^^^^^^^^^^^^^^^^^^^
report
midrule.y:2.6-8: warning: unset value: $$
exp: {;} {$$;} { $$ = $1; }
^^^
midrule.y:2.10-14: warning: unused value: $2
exp: {;} {$$;} { $$ = $1; }
^^^^^
* src/reader.c (grammar_rule_check): When warning about the value of a
midrule action, use the location of the midrule action instead of the
location of the rule.
the location of the part of the rule.
* tests/actions.at (Default %printer and %destructor for mid-rule values):
Adjust expectations
* tests/input.at (Unused values with default %destructor): Ditto.
(AT_CHECK_UNUSED_VALUES): Ditto.
And use -fcaret.
Akim Demaille [Sat, 8 Dec 2012 15:18:07 +0000 (16:18 +0100)]
doc: various minor improvements and fixes
* doc/figs/example.dot, doc/figs/example.y: New.
* doc/bison.texi: Prefer "token" to TOKEN.
Use @group where appropriate.
Adjust with style changes in the output (State 0, not state 0).
Fix some @ref that were missing the third argument.
Fix some incorrect line numbers.
Use "nonterminal", not "non-terminal".
Fix overfull and underfull TeX hboxes.
Put the comments in the index.
Remove duplicate index entries.
Fuse glossary entries where appropriate.
(Understanding): Improve the continuity between sections.
Use example.dot to show the whole graph.
* doc/Makefile.am: Adjust.
Akim Demaille [Sun, 9 Dec 2012 15:23:34 +0000 (16:23 +0100)]
tests: ignore more useless compiler warnings
* tests/synclines.at (AT_SYNCLINES_COMPILE): Ignore complains about
using c++ to compile C.
Akim Demaille [Sun, 9 Dec 2012 14:41:50 +0000 (15:41 +0100)]
tests: be robust to C being compiled with a C++11 compiler
* tests/glr-regression.at: Use YY_NULL instead of NULL.
Comment changes.
Akim Demaille [Sat, 8 Dec 2012 18:33:50 +0000 (19:33 +0100)]
build: beware of Clang++ not supporting POSIXLY_CORRECT
* m4/c-working.m4 (BISON_LANG_COMPILER_POSIXLY_CORRECT): New.
(BISON_C_COMPILER_POSIXLY_CORRECT): Use it.
For consistency with C++, also define BISON_C_WORKS.
* m4/cxx.m4 (BISON_CXX_COMPILER_POSIXLY_CORRECT): New.
* configure.ac: Use it.
* tests/atlocal.in: Get its result.
Propagate properly CXX values when used to compile C.
When POSIXLY_CORRECT, adjust BISON_C_WORKS and BISON_CXX_WORKS.
* tests/local.at (AT_COMPILE): Use BISON_C_WORKS.
Akim Demaille [Fri, 7 Dec 2012 10:49:12 +0000 (11:49 +0100)]
maint: post-release administrivia
* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.
Akim Demaille [Fri, 7 Dec 2012 10:37:14 +0000 (11:37 +0100)]
version 2.6.90
* NEWS: Record release date.
Akim Demaille [Fri, 7 Dec 2012 10:34:51 +0000 (11:34 +0100)]
build: fix syntax-check error.
* cfg.mk: Exclude names-refs, it includes a "double" if (end of first
line, first of second line below).
test.y:43.12-44.59: symbol not found in production: if
if-stmt-a: IF expr[cond] THEN stmt.list[then] ELSE stmt.list[else] FI
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Theophile Ranquet [Fri, 7 Dec 2012 11:13:38 +0000 (12:13 +0100)]
cpp: simplify the Flex version checking macro
* src/flex-scanner,h (FLEX_VERSION): Consider YY_FLEX_SUBMINOR_VERSION
defined.
Theophile Ranquet [Fri, 7 Dec 2012 10:57:19 +0000 (11:57 +0100)]
news: improve the carets example and fix a typo
* NEWS: Here.
Theophile Ranquet [Thu, 6 Dec 2012 12:21:36 +0000 (13:21 +0100)]
cpp: improve the Flex version checking macro
* src/flex-scanner.h (FLEX_VERSION): Here.
Theophile Ranquet [Thu, 6 Dec 2012 12:17:55 +0000 (13:17 +0100)]
carets: improve the code
* src/location.c: Remove duplicate documentations.
(caret_info): Stylistic change.
(location_caret): Many reworks.
Akim Demaille [Fri, 7 Dec 2012 09:40:31 +0000 (10:40 +0100)]
maint: update news
* NEWS: There is no 2.6.6, remove its stub.
Akim Demaille [Fri, 7 Dec 2012 08:58:40 +0000 (09:58 +0100)]
build: keep -Wmissing-declarations and -Wmissing-prototypes for modern GCCs
Fixes a -Werror failure of xalloc.h used in src.
From Eric Blake.
http://lists.gnu.org/archive/html/bug-gnulib/2012-12/msg00006.html
* configure.ac: Check whether GCC pragma diagnostic push/pop works.
Enable these warnings for bison if it does.
Enable these warnings for the test suite anyway.
Akim Demaille [Fri, 7 Dec 2012 08:48:41 +0000 (09:48 +0100)]
build: drop -Wcast-qual
Suggested by Jim Meyering.
http://lists.gnu.org/archive/html/bug-gnulib/2012-12/msg00017.html
* configure.ac (warn_common): Remove -Wcast-qual.
Akim Demaille [Wed, 5 Dec 2012 15:16:17 +0000 (16:16 +0100)]
gnulib: update
Theophile Ranquet [Thu, 6 Dec 2012 10:43:02 +0000 (11:43 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
misc: pacify the Tiny C Compiler
cpp: make the check of Flex version portable
misc: require getline
c++: support wide strings for file names
doc: document carets
tests: enhance existing tests with carets
errors: show carets
getargs: add support for --flags/-f
Conflicts:
doc/bison.texi
m4/.gitignore
src/complain.c
src/flex-scanner.h
src/getargs.c
src/getargs.h
src/gram.c
src/main.c
tests/headers.at
Theophile Ranquet [Thu, 6 Dec 2012 09:49:12 +0000 (10:49 +0100)]
misc: pacify the Tiny C Compiler
* src/graphviz.c (conclude_red): Remove a useless return.
Theophile Ranquet [Wed, 5 Dec 2012 17:26:36 +0000 (18:26 +0100)]
cpp: make the check of Flex version portable
This was problematic with tcc 0.9.25
* src/flex-scanner.h (FLEX_VERSION_GT): Rewrite and rename as...
(FLEX_VERSION): This.
Theophile Ranquet [Wed, 5 Dec 2012 14:27:25 +0000 (15:27 +0100)]
misc: require getline
* bootstrap.conf: Here, used by src/location.c.
* src/getargs.c (long_options): Rename --flags to --feature.
Akim Demaille [Wed, 5 Dec 2012 10:21:21 +0000 (11:21 +0100)]
c++: support wide strings for file names
Reported by Mark Boyall.
http://lists.gnu.org/archive/html/help-bison/2011-08/msg00002.html
* data/location.cc (operator<<): Be templated on the type of
output stream.
* tests/headers.at (Several parsers): Adjust.
Theophile Ranquet [Tue, 4 Dec 2012 12:12:12 +0000 (13:12 +0100)]
doc: document carets
* NEWS: Announce it.
* doc/bison.texi (Bison Options): Here.
Theophile Ranquet [Mon, 3 Dec 2012 10:12:07 +0000 (11:12 +0100)]
tests: enhance existing tests with carets
* tests/actions.at: Unset value.
* tests/conflicts.at: Rule useless due to conflicts.
* tests/input.at: Missing terminator, unexpected end of file, command line
redefinition of variable.
* tests/named-refs.at: Many errors.
* tests/reduce.at: Useless nonterminals and rules.
* tests/regression.at: Large token.
Theophile Ranquet [Fri, 30 Nov 2012 13:34:56 +0000 (14:34 +0100)]
errors: show carets
* src/locations.c (caret_info): New, persistant information useful
for...
(location_caret): New, print a caret.
(cleanup_caret): Release caret_info cleanly, call it...
* src/main.c (main): Here.
* src/complain.c (error_message): Call location_caret here.
Theophile Ranquet [Fri, 30 Nov 2012 13:33:05 +0000 (14:33 +0100)]
getargs: add support for --flags/-f
Introduce -fdiagnostics-show-caret
* src/getargs.c (flag_flag): New global.
* src/getargs.h (flag): New enum.
Akim Demaille [Wed, 5 Dec 2012 10:03:08 +0000 (11:03 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
getargs: don't label --language/-l as experimental
getargs: fix the locations of command-line input
errors: indent missing action code semicolon warning
Conflicts:
NEWS
src/scan-code.l
tests/actions.at
Theophile Ranquet [Fri, 30 Nov 2012 14:27:54 +0000 (15:27 +0100)]
getargs: don't label --language/-l as experimental
* NEWS: Announce it.
* doc/bison.texi, src/getargs.c (usage): Here.
Akim Demaille [Mon, 3 Dec 2012 16:27:54 +0000 (17:27 +0100)]
tests: minor improvements
* tests/calc.at (AT_CHECK_SPACES): To speed up, accept several files
at once, and factor some calls.
Theophile Ranquet [Mon, 12 Mar 2012 09:01:09 +0000 (10:01 +0100)]
getargs: fix the locations of command-line input
* src/getargs.c (command_line_location): Here.
* tests/input.at: Adjust.
Theophile Ranquet [Thu, 15 Nov 2012 12:02:40 +0000 (12:02 +0000)]
errors: indent missing action code semicolon warning
Also, remove a duplicate #define.
* src/scan-code.l (SC_RULE_ACTION): Here.
* tests/actions.at: Adjust.
Akim Demaille [Mon, 3 Dec 2012 15:27:23 +0000 (16:27 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
parser: accept #line NUM
m4: use a safer pattern to enable/disable output
tests: beware of gnulib's need for config.h
gnulib: update
yacc.c, glr.c: check and fix the display of locations
formatting changes
glr.c: remove stray macro
Conflicts:
data/c.m4
data/glr.cc
data/lalr1.cc
data/lalr1.java
data/location.cc
data/stack.hh
data/yacc.c
src/scan-gram.l
Akim Demaille [Mon, 3 Dec 2012 14:59:57 +0000 (15:59 +0100)]
parser: accept #line NUM
* src/scan-gram.l (scanner): Accept '#line NUM'.
(handle_syncline): Adjust to the possible missing file name.
Akim Demaille [Mon, 3 Dec 2012 14:29:44 +0000 (15:29 +0100)]
m4: use a safer pattern to enable/disable output
Work on some other areas of Bison revealed that some macros expanded
to be expanded only once were actually expanded several times. This
was due to the fact that changecom was not properly restored each
time, and macro names appearing in comments were then expanded.
Introduce begin/end macros which are easier to match that
changecom()/changecom(#).
* data/bison.m4 (b4_output_begin, b4_output_end): New.
* data/glr.c, data/glr.cc, data/lalr1.cc, data/lalr1.java,
* data/location.cc, data/stack.hh, data/yacc.c:
Use them.
Akim Demaille [Mon, 3 Dec 2012 14:43:57 +0000 (15:43 +0100)]
tests: beware of gnulib's need for config.h
* tests/skeletons.at, tests/torture.at: Be sure to include config.h
where appropriate.
Akim Demaille [Fri, 30 Nov 2012 11:17:10 +0000 (12:17 +0100)]
gnulib: update
* lib/yyerror.c: Include config.h since the following stdio.h might be
from gnulib.
Akim Demaille [Fri, 30 Nov 2012 10:19:43 +0000 (11:19 +0100)]
yacc.c, glr.c: check and fix the display of locations
In some case, negative column number could be displayed.
Make YY_LOCATION_PRINT similar to bison's own implementation of
locations. Since the macro is getting fat, make it a static
function.
Reported by Jonathan Fabrizio.
* data/c.m4 (yy_location_print_define): Improve the implementation,
and generate the yy_location_print_ function.
Adjust YY_LOCATION_PRINT.
* tests/actions.at (Location Print): New tests.
Akim Demaille [Fri, 30 Nov 2012 10:19:33 +0000 (11:19 +0100)]
formatting changes
* data/c.m4: Fix comments, put macros in a more natural order.
Space changes (from M-x whitespace-cleanup).
* src/location.c: Fix spaces.
* tests/actions.at: Space changes.
Akim Demaille [Fri, 30 Nov 2012 10:15:12 +0000 (11:15 +0100)]
glr.c: remove stray macro
* data/glr.c (YYOPTIONAL_LOC): Remove, unused since commit
769a8ef9bcb5e14d0be9d0869f5dca20ab093930.
Akim Demaille [Thu, 29 Nov 2012 13:54:37 +0000 (14:54 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
doc: minor fixes
doc: improve the index
doc: introduce api.pure full, rearrange some examples
yacc.c: support "%define api.pure full"
local.at: improvements
Conflicts:
NEWS
data/yacc.c
doc/bison.texi
tests/calc.at
Akim Demaille [Thu, 29 Nov 2012 13:35:48 +0000 (14:35 +0100)]
doc: minor fixes
* doc/bison.texi: Use stderr for error messages.
Meta-variables are usually spelled in lower case.
Use @code for function names.
Akim Demaille [Thu, 29 Nov 2012 13:09:34 +0000 (14:09 +0100)]
doc: improve the index
* doc/bison.texi: Fix uses of "deffn" so that the arguments of the
directives do not show in the index.
Remove a duplicate entry for api.pure.
Theophile Ranquet [Mon, 26 Nov 2012 17:17:15 +0000 (18:17 +0100)]
doc: introduce api.pure full, rearrange some examples
* NEWS: Add entry.
* doc/bison.texi (%define Summary): Show the old Yacc behaviour.
(Parser Function): Move parse-param examples here.
(Pure Calling): Remove parse-param examples.
(Error Reporting): Don't show the old behavior, stick to 'full'.
Theophile Ranquet [Fri, 23 Nov 2012 17:30:47 +0000 (17:30 +0000)]
yacc.c: support "%define api.pure full"
This makes the interface for yyerror() pure without the need for a spurious
parse_param.
* data/yacc.c (b4_pure_if, b4_pure_flag): New definition, accept three states.
(b4_yacc_pure_if): Rename as...
(b4_yyerror_arg_loc_if): This, and use b4_pure_flag.
* tests/actions.at (%define api.pure): Modernize.
* test/calc.at (Simple LALR Calculator): Modernize.
* tests/local.at (AT_YYERROR_ARG_LOC_IF): Adjust.
Akim Demaille [Wed, 28 Nov 2012 09:51:45 +0000 (10:51 +0100)]
tests: check variants without locations
* tests/c++.at (Variants): Support non-use of locations, and
check its support.
Theophile Ranquet [Fri, 23 Nov 2012 16:23:27 +0000 (16:23 +0000)]
local.at: improvements
* tests/local.at (AT_YYERROR_FORMALS): Make llocp const.
(AT_PURE_AND_LOC_IF, AT_GLR_OR_PARAM_IF): Remove, expand...
(AT_YYERROR_ARG_LOC_IF): Here, and use m4_join for readability.
Akim Demaille [Mon, 26 Nov 2012 09:42:49 +0000 (10:42 +0100)]
tests: use -fno-strict-aliasing with variants
Reported by Théophile Ranquet.
* configure.ac (NO_STRICT_ALIAS_CXXFLAGS): New.
* tests/c++.at, tests/atlocal.in, examples/local.mk: Use it.
Akim Demaille [Mon, 26 Nov 2012 09:42:43 +0000 (10:42 +0100)]
tests: remove leftover
* tests/atlocal.in: Remove duplicate handling of --compile-c-with-cxx.
Akim Demaille [Mon, 26 Nov 2012 08:49:23 +0000 (09:49 +0100)]
doc: use %precedence instead of nonassoc when associativity is not wanted
* doc/bison.texi: here.
Formatting changes in some grammars.
Fix a %prec into %precedence.
Akim Demaille [Mon, 26 Nov 2012 08:14:51 +0000 (09:14 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
yacc.c: always initialize yylloc
scanner: issue a single error for groups of invalid characters
tests: formatting changes
doc: one of the fixes for an ambiguous grammar was ambiguous too
doc: fix the dangling else with precedence directives
doc: prefer "token" to TOKEN
doc: formatting changes
scanner: use explicit "ignore" statements
Conflicts:
src/scan-gram.l
Akim Demaille [Mon, 26 Nov 2012 08:05:28 +0000 (09:05 +0100)]
Merge remote-tracking branch 'origin/branch-2.6' into maint
* origin/branch-2.6:
yacc.c: always initialize yylloc
doc: one of the fixes for an ambiguous grammar was ambiguous too
doc: fix the dangling else with precedence directives
doc: prefer "token" to TOKEN
doc: formatting changes
Conflicts:
NEWS
doc/bison.texi
Theophile Ranquet [Mon, 19 Nov 2012 10:43:56 +0000 (10:43 +0000)]
yacc.c: always initialize yylloc
The initial location might be used if the parser starts by an empty
reduction, so really ensure proper initialization of the initial
location. The previous approach fails for PostgreSQL, which uses
Reported by Peter Eisentraut.
http://lists.gnu.org/archive/html/bug-bison/2012-11/msg00023.html
With help from Théophile Ranquet.
* data/yacc.c (b4_declare_scanner_communication_variables): Be sure
to initialize yylloc, even when its structure is unknown.
(yyparse): Simplify the call to b4_dollar_pushdef.
* tests/actions.at (Initial location): Check of similar pattern
as in the case of PostgreSQL.
Akim Demaille [Fri, 23 Nov 2012 10:21:47 +0000 (11:21 +0100)]
scanner: issue a single error for groups of invalid characters
* src/scan-gram.l: Scan groups of invalid characters together.
* tests/input.at, tests/named-refs.at: Adjust.
Akim Demaille [Fri, 23 Nov 2012 09:11:50 +0000 (10:11 +0100)]
tests: formatting changes
* tests/named-refs.at: Here.
Akim Demaille [Thu, 22 Nov 2012 14:49:18 +0000 (15:49 +0100)]
doc: one of the fixes for an ambiguous grammar was ambiguous too
Reported by Аскар Сафин.
http://lists.gnu.org/archive/html/bug-bison/2012-11/msg00024.html
* doc/bison.texi (Reduce/Reduce): Fix the resulting ambiguity using
precedence/associativity directives.
Akim Demaille [Thu, 22 Nov 2012 14:16:07 +0000 (15:16 +0100)]
doc: fix the dangling else with precedence directives
* doc/bison.texi (Non Operators): New node.
(Shift/Reduce): Point to it.
Don't promote "%expect n" too much.
Akim Demaille [Thu, 22 Nov 2012 14:12:24 +0000 (15:12 +0100)]
doc: prefer "token" to TOKEN
This is more readable in short examples.
* doc/bison.texi (Shift/Reduce): here.
Make "win" and "lose" action more alike.
Akim Demaille [Thu, 22 Nov 2012 13:24:54 +0000 (14:24 +0100)]
doc: formatting changes
* doc/bison.texi: Use @group.
Akim Demaille [Wed, 14 Nov 2012 09:57:45 +0000 (10:57 +0100)]
scanner: use explicit "ignore" statements
* src/scan-gram.l: here.
Akim Demaille [Tue, 13 Nov 2012 09:59:55 +0000 (10:59 +0100)]
Merge remote-tracking branch 'origin/maint'
* origin/maint:
tests: close files in glr-regression
xml: match DOT output and xml2dot.xsl processing
xml: factor xslt space template
graph: fix a memory leak
xml: documentation
output: capitalize State
Theophile Ranquet [Mon, 12 Nov 2012 15:41:52 +0000 (15:41 +0000)]
tests: close files in glr-regression
* tests/glr-regression.at: Here.
Theophile Ranquet [Tue, 23 Oct 2012 15:43:54 +0000 (15:43 +0000)]
xml: match DOT output and xml2dot.xsl processing
Make the DOT produced by XSLT processing equivalent to the one made with the
--graph option.
* data/xslt/xml2dot.xsl: Stylistic changes, and add support for reductions.
* doc/bison.texi (Xml): Update.
* src/graphviz.c (conclude_red): Minor stylistic changes to DOT internals.
(output_red): Swap enabled and disabled reductions output, for coherence
with XSLT output.
* src/print_graph.c (print_core): Minor stylistic change to States' output.
(print_actions): Swap order of output for reductions and transitions.
* tests/local.at (AT_BISON_CHECK_XML): Ignore differences in order.
* tests/output.at: Adjust to changes in DOT internals.
Theophile Ranquet [Wed, 7 Nov 2012 09:54:06 +0000 (09:54 +0000)]
xml: factor xslt space template
* data/xslt/bison.xsl (space): New, import from...
* data/xslt/xml2text.xsl: Here.
Theophile Ranquet [Fri, 9 Nov 2012 16:40:45 +0000 (16:40 +0000)]
graph: fix a memory leak
* src/graphviz.c (output_red): Here.
Theophile Ranquet [Mon, 22 Oct 2012 17:37:57 +0000 (17:37 +0000)]
xml: documentation
The XML output combined with the XSL Transformations provided in data/ are
incredibly useful, they should be documented.
* doc/bison.texi (Xml): New node.