Skip to content

Don't lookahead from interpolation#25834

Merged
som-snytt merged 1 commit intoscala:mainfrom
som-snytt:issue/25717-interpol-assert
Apr 20, 2026
Merged

Don't lookahead from interpolation#25834
som-snytt merged 1 commit intoscala:mainfrom
som-snytt:issue/25717-interpol-assert

Conversation

@som-snytt
Copy link
Copy Markdown
Contributor

Fixes #25717

Verified manually that other usages of lookahead check the current in.token first.

How much have you relied on LLM-based tools in this contribution?

Not at all

How was the solution tested?

New automated tests (including the issue's reproducer, if applicable)

@som-snytt som-snytt marked this pull request as ready for review April 17, 2026 07:18
@Gedochao Gedochao assigned Gedochao and warcholjakub and unassigned Gedochao Apr 17, 2026
Copy link
Copy Markdown
Member

@warcholjakub warcholjakub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@som-snytt som-snytt merged commit 2c89135 into scala:main Apr 20, 2026
45 checks passed
@som-snytt som-snytt deleted the issue/25717-interpol-assert branch April 20, 2026 12:41
mbovel pushed a commit to mbovel/dotty that referenced this pull request May 4, 2026
Fixes scala#25717 

Verified manually that other usages of `lookahead` check the current
`in.token` first.
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.

AssertionError during lexical analysis of a macro definition

3 participants