search

canonical / sqlair

Friendly type mapping for SQL databases
Apache License 2.0
16 stars 8 forks source link

simplify parser #137

Closed letFunny closed 3 months ago

letFunny commented 5 months ago

This commits helps reason about the parser and in adding recursive parsing latter. The main idea is the separation of concern between expression parsing, adding bypass parts and the advance() function; all three were interleaved before.

letFunny commented 5 months ago

@Aflynn50 Agree on the tests, I was trying to publish this as a draft PR to discuss it and then add the tests once we agreed on what to include.