-
Notifications
You must be signed in to change notification settings - Fork 728
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add syntax rewriter framework and some decompiler simplification rewriters #953
Conversation
Codecov Report
@@ Coverage Diff @@
## main #953 +/- ##
==========================================
- Coverage 94.33% 94.24% -0.09%
==========================================
Files 316 326 +10
Lines 14771 15575 +804
Branches 12 12
==========================================
+ Hits 13934 14679 +745
- Misses 837 896 +59
Flags with carried forward coverage won't be shown. Click here to find out more.
|
You have an empty file called InvalidOperationException.cs btw. |
return syntax; | ||
} | ||
|
||
// TODO what to do with lexerDiagnostics? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lexerDiagnostics [](start = 36, length = 16)
This makes me wonder if we need to have errors on tokens that get collected like parse errors. (Most tokens don't have errors so it could also be a new ErrorToken class derived from Token to avoids allocating extra space for mostly null pointers.)
Thanks for the heads-up :) Omnisharp makes it annoyingly easy to accidentally do that... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'{parent.name}/child'
style syntaxCloses [Decompilation] Simplifications to decompilation output #965
Closes [Decompilation] Automatically fix parent-child resource name dependencies #964
Preview these changes on the quickstart repo here.