Skip to content

Conversation

@odersky
Copy link
Contributor

@odersky odersky commented Nov 11, 2025

We now treat update like erased and consume. The error message is still not great, maybe something can be done about that.

We now treat update like erased and consume. The error message is still not great,
maybe something can be done about that.
@hamzaremmal
Copy link
Member

Relates to #24371

Copy link
Member

@hamzaremmal hamzaremmal left a comment

Choose a reason for hiding this comment

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

Note that this work because we revive a dead path in the Scanner.

def isSoftModifier: Boolean =
token == IDENTIFIER
&& (softModifierNames.contains(name)
|| name == nme.erased && erasedEnabled
|| name == nme.tracked && trackedEnabled
|| name == nme.update && Feature.ccEnabled
|| name == nme.consume && Feature.ccEnabled)

Before, we matched in line 1230 but now we will match in line 1233.

@hamzaremmal hamzaremmal merged commit b1c4e05 into scala:main Nov 12, 2025
87 of 88 checks passed
@hamzaremmal hamzaremmal deleted the fix-24371 branch November 12, 2025 12:46
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.

2 participants