-
Notifications
You must be signed in to change notification settings - Fork 48
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
Fix preprocessor expansion strategy #33
Merged
Merged
Commits on Oct 3, 2021
-
Preprocessor: redesign the expansion strategy for macros
The previous macro expansion implementation had several flaws regarding order and re-expansion logic. Those were fixed by a complete rewrite of the expandMacro() and its nested code paths. Also remove special handling for 'self' macros. In the future, the 'empty' macro will be removed as well.
Configuration menu - View commit details
-
Copy full SHA for fec07ca - Browse repository at this point
Copy the full SHA fec07caView commit details -
Configuration menu - View commit details
-
Copy full SHA for 201abc5 - Browse repository at this point
Copy the full SHA 201abc5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3cbf372 - Browse repository at this point
Copy the full SHA 3cbf372View commit details -
Preprocessor: emulate gcc by not rescanning completely expanded token…
…s at beginning These modifications strive for compatibility with gcc, but I'm not sure it is strictly mandated by the standard
Configuration menu - View commit details
-
Copy full SHA for c71e994 - Browse repository at this point
Copy the full SHA c71e994View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4b6f68b - Browse repository at this point
Copy the full SHA 4b6f68bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6b54645 - Browse repository at this point
Copy the full SHA 6b54645View commit details -
Configuration menu - View commit details
-
Copy full SHA for 498c30a - Browse repository at this point
Copy the full SHA 498c30aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1c9469d - Browse repository at this point
Copy the full SHA 1c9469dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 723cbd8 - Browse repository at this point
Copy the full SHA 723cbd8View commit details
Commits on Oct 6, 2021
-
Configuration menu - View commit details
-
Copy full SHA for 6544dcf - Browse repository at this point
Copy the full SHA 6544dcfView commit details -
Preprocessor: apply suggestions from code review
Mainly changes to the control flow of the macro expansion/rescan loop Co-authored-by: Veikka Tuominen <git@vexu.eu>
Configuration menu - View commit details
-
Copy full SHA for bedd6ac - Browse repository at this point
Copy the full SHA bedd6acView commit details -
Configuration menu - View commit details
-
Copy full SHA for 31449f4 - Browse repository at this point
Copy the full SHA 31449f4View commit details -
Configuration menu - View commit details
-
Copy full SHA for e3e4ac4 - Browse repository at this point
Copy the full SHA e3e4ac4View commit details
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.