-
Notifications
You must be signed in to change notification settings - Fork 2
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
refactor!: change lexer to use acorn and escodegen
- refactor!: change lexer to use acorn and codegen instead of js-tokens. Realized that creating our own AST reader is a huge task, make take a lot of time and prone for errors if we want to be spec compliant. With this, I decided to use acorn and codegen instead in terms of identifying variables. - fix: non-MiniJS variables are being read by Mini.js. I made so that we are not automatically reading user defined global variables. We now only fetch variables from attributes and events. Currently, we can't fetch from the scripts since we are not sure whether that script is defined by a user or a library. - fix: not being able to declare variables due to being read as a proxy variable.
- Loading branch information
Showing
6 changed files
with
366 additions
and
371 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.