Added in Symbol Tables #2
Merged
+1,566
−774
Conversation
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
…private renaming methods
…nality of the file
…tedly This changes it so every file will only be read from disk once, instead of being read three different times This comes at a memory tradeoff, but the total cost of each source file is extremely small, even for a large project, that this is a large speedup
…pe definition names Search now works off of the original types of the method
…ructors with generic types
…okups in static blocks
…variables and methods
* Separated out some functionality away from both the symbols and parsing * Major clean-up for some of the parsing code
Moved over some more of the declaration logic to use the new syntax
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR merges in the functionality to use symbol tables by default. This means things such as
Any many more things are handled more correctly this time, bringing the translation closer to a 1-to-1 conversion
This will however, make some parts of the parser more fragile, so there has been a flag added to disable this functionality, which is in-progress