akirschbaum

firstOf() fails for one-character String argument

1

When using firstOf() with only String arguments and if one String has length 1, then parser generation fails with a reference to trie(). This is incorrect for two reasons: I...

firstOf() ignores overridden fromStringLiteral(String)

1

The following grammar contains three rules. Each of these rules should match inputs "a" or "b", optionally followed by spaces. rule1 explicitly matches the trailing whitespace with zeroOrMore(sp()). This rule...

anyOf(Characters.ALL) does match even if no more input is available. It should not match because no input character is available and thus no input character could have been consumed. The...

Static rule creates broken parser

1

A rule that is declared "static" causes a "java.lang.VerifyError" during parser generation. Instead Grappa should either support this or else should generate a proper error message. ``` $ gradle :compileJava...

Recursive rules passing different Var instances should be possible

1

When using rules that have parameters, @Cached can be used to allow recursive calls. The generated caching code then checks all parameters for equality when reusing rules. This mechanism works...

Allow custom parse error messages

1

It should be possible to generate custom error messages from a parser. For example with a function like _BaseParser.parseError(String message)_ which causes a parse error with the given custom error...

Spurious compiler warnings due to @Nullable annotations

2

Recent Grappa versions annotate _Var.get()_ with _@Nullable_. While technically correct, this causes spurious warnings in parsers: ``` Rule r() { Var v = new Var(); return sequence('x', v.set("x"), 'y', f(v.get()));...

Allow parse warning messages

1

It should be possible to issue warning messages from the parser. These messages should be available from _ParsingResult_ but should not count as parsing errors.