search

crashappsec / libcon4m

Base Compiler and Runtime Support for con4m
Apache License 2.0
0 stars 0 forks source link

Parser "port" #16

Closed viega closed 4 months ago

viega commented 4 months ago

The parser is ported to the point where all of the old test files give the parse trees I expect (validated manually).

It's different enough that I'd expect some lingering issues.

The biggest thing that needs improvement is the error handling. It isn't as good as the Nim version (which also wasn't as good as I wanted it).

Specifically:

  1. Error recovery isn't awesome. I'm just doing a lot of line skipping instead of token insertion / deletion, etc.
  2. I'm actually skipping EXTRA lines at this point.
  3. I bail out of the whole thing when there was any complexity at all.
  4. I'm not doing the niceties I was in Nim yet, like underlying / highlighting.

No hurry on these items though.