Skip to content
This repository was archived by the owner on Sep 10, 2025. It is now read-only.

Improve usability in our parser - #87

Merged
skx merged 2 commits into
masterfrom
86-parser
Jan 17, 2022
Merged

skx merged 2 commits into
masterfrom
86-parser

Conversation

@skx

@skx skx commented Jan 17, 2022

Copy link
Copy Markdown
Owner

This pull-request will close #86 by preventing "{" and "}" from being considered valid characters in identifers.

This has the net result that shell{command... will be parsed correctly, whereas previously it would have resulted in a parse failure as that would have been turned into the identifier with name shell{command - which would then have triggered a syntax-error as the expected { wouldn't have been found to follow that identifier.

@skx skx self-assigned this Jan 17, 2022
@skx
skx merged commit 386ddba into master Jan 17, 2022
@skx
skx deleted the 86-parser branch January 17, 2022 19:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

It shouldn't be necessary to separate module from "{"

1 participant

⌂ 换网址