I tested these two patches through the test - Unit Tests
which has 44 test cases and all passed.
[1/2] modify the g4 and parser.cs
[2/2] modify all between () -> between .. and in the tests.
In fact I don't have time to read the ANTLR book, and I am not quite sure if this modification is the right/best way, for example ,
should I separate expr into logical_expr, bool_expr and arith_expr, I don't have theory support but just lots of tries.
I use three ( .. ) in logical_expr, arith_expr, and expr, I am sure the expr one should never entered and I will leave an assertion there, when I figure out how to modify this:
public override object VisitOtherbrackexpr([NotNull] SQLiteParser.OtherbrackexprContext context)
=> Visit(context.expr());
I tested these two patches through the test - Unit Tests which has 44 test cases and all passed.
[1/2] modify the g4 and parser.cs [2/2] modify all between () -> between .. and in the tests.
In fact I don't have time to read the ANTLR book, and I am not quite sure if this modification is the right/best way, for example ,
public override object VisitOtherbrackexpr([NotNull] SQLiteParser.OtherbrackexprContext context) => Visit(context.expr());
-.-|||