Skip to content

Conversation

@tlively
Copy link
Member

@tlively tlively commented May 18, 2024

Remove SExpressionParser, SExpressionWasmBuilder, and cashew::Parser.
Simplify gen-s-parser.py. Remove the --new-wat-parser and
--deprecated-wat-parser flags.

@tlively tlively requested a review from kripken May 18, 2024 01:50
Copy link
Member Author

tlively commented May 18, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tlively and the rest of your teammates on Graphite Graphite

extern bool isIdentInit(char x);
extern bool isIdentPart(char x);

// parser
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This parses JS, which I guess could be a separate PR. But maybe not worth it if it's just a safe removal.

@tlively tlively force-pushed the wasm2js-rewrite branch from 99ac5e2 to 6ca0bfa Compare May 29, 2024 02:10
Base automatically changed from wasm2js-rewrite to main May 29, 2024 17:35
tlively added 2 commits May 29, 2024 10:54
Remove `SExpressionParser`, `SExpressionWasmBuilder`, and `cashew::Parser`.
Simplify gen-s-parser.py. Remove the --new-wat-parser and
--deprecated-wat-parser flags.
@tlively tlively force-pushed the delete-old-parser branch from dfb35eb to 5973864 Compare May 29, 2024 18:03
@tlively tlively merged commit d844d2e into main May 29, 2024
@tlively tlively deleted the delete-old-parser branch May 29, 2024 18:32
@gkdn gkdn mentioned this pull request Aug 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants