-
-
Notifications
You must be signed in to change notification settings - Fork 185
Move parentheses usage logic from AST to printer #665
Copy link
Copy link
Closed
Description
ark120202
opened on Jul 14, 2019
Issue body actions
- Having specific nodes for parentheses doesn't make much sense for me. Parentheses are supposed to be a tip to parser, in AST we don't have any ambiguities where parentheses matter:
- It would make code cleaner and easier to maintain, since to fix issues like Fix for property access on string and array literals #650 we won't have to duplicate logic everywhere
- It would make code less bug-prone, since we'll have less invalid states in AST
- Most of JS parsers (babel, acorn, esprima) do it
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels