bison-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 0/9] Prepare Java for parse.error custom|detailed


From: Akim Demaille
Subject: [PATCH 0/9] Prepare Java for parse.error custom|detailed
Date: Sun, 2 Feb 2020 15:26:18 +0100

The Calculator test suite for Java is too different from the main one,
which would result is too much test duplication when extended to
support the new forms of error reporting.  So I wanted to first clean
up the test suite, but it quickly revealed a number of shortcomings:

- locations tracks the token number in the current line, instead
  of the actual column number.  To address this, I had to do more
  Java than I ever did before...

- the tests were still dating from before autoboxing.

- trailing spaces.

and other minor issues that were in the way.

I still have to finish migrating the calculator test suite from
java.at to calc.at, but also to split the Java example in two: a
simple one without location tracking, and a more complete one.

Akim Demaille (9):
  skeletons: add support for %code epilogue
  tests: comment changes
  java: example: rely on autoboxing
  java: example: improve
  java: example: properly track the locations
  java: avoid trailing white spaces
  java: formatting changes
  java: add access to the number of errors
  java: use the same calc tests as the other skeletons

 data/skeletons/glr.c      |   3 +-
 data/skeletons/java.m4    |  16 +-
 data/skeletons/lalr1.d    |   5 +-
 data/skeletons/lalr1.java | 142 ++++++++--------
 data/skeletons/yacc.c     |   3 +-
 examples/java/Calc.test   |   9 +-
 examples/java/Calc.y      | 110 +++++++++----
 tests/calc.at             | 331 +++++++++++++++++++++++++++++++-------
 tests/java.at             |  42 ++---
 tests/javapush.at         |   4 +-
 tests/local.at            |  11 +-
 11 files changed, 472 insertions(+), 204 deletions(-)

-- 
2.25.0




reply via email to

[Prev in Thread] Current Thread [Next in Thread]