On Wed, Mar 20, 2024 at 11:56 PM Andrew Dunstan <andrew@dunslane.net> wrote: > Thanks, included that and attended to the other issues we discussed. I think this is pretty close now.
Okay, looking over the thread, there are the following open items: - extend the incremental test in order to exercise the semantic callbacks [1]
Yeah, I'm on a super long plane trip later this week, so I might get it done then :-)
- add Assert calls in impossible error cases [2]
ok, will do
- error out if the non-incremental lex doesn't consume the entire token [2]
ok, will do
- double-check that out of memory is an appropriate failure mode for the frontend [3]
Well, what's the alternative? The current parser doesn't check stack depth in frontend code. Presumably it too will eventually just run out of memory, possibly rather sooner as the stack frames could be more expensive than the incremental parser stack extensions.