Skip to content
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 Splicer.isEscapedVariable #16838

Merged
merged 1 commit into from
Feb 16, 2023
Merged

Fix Splicer.isEscapedVariable #16838

merged 1 commit into from
Feb 16, 2023

Conversation

nicolasstucki
Copy link
Contributor

Consider that val macro expansion in the context can come from an outer macro that is being expanded (i.e. this is a nested macro). Nested macro expansion can occur when a macro summons an implicit macro.

Fixes partially #16835

Consider that `val macro` expansion in the context can come from
an outer macro that is being expanded (i.e. this is a nested macro).
Nested macro expansion can occur when a macro summons an implicit macro.

Fixes partially scala#16835
@nicolasstucki
Copy link
Contributor Author

Resolved a small conflict with another PR that also added isMacroOwner

@nicolasstucki nicolasstucki added this pull request to the merge queue Feb 14, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 15, 2023
@nicolasstucki nicolasstucki added this pull request to the merge queue Feb 15, 2023
@Kordyjan Kordyjan removed this pull request from the merge queue due to the queue being cleared Feb 15, 2023
@Kordyjan Kordyjan added this pull request to the merge queue Feb 15, 2023
@Kordyjan Kordyjan removed this pull request from the merge queue due to the queue being cleared Feb 15, 2023
@Kordyjan Kordyjan added this pull request to the merge queue Feb 15, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Feb 15, 2023
@Kordyjan Kordyjan merged commit 60f2b96 into scala:main Feb 16, 2023
@Kordyjan Kordyjan deleted the fix-16835 branch February 16, 2023 09:02
@Kordyjan Kordyjan added this to the 3.3.1 milestone Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants