Skip to content

Remove workaround for $outer not marked synthetic#354

Closed
Philippus wants to merge 1 commit intospray:release/1.3.xfrom
Philippus:remove-synthetic-workaround
Closed

Remove workaround for $outer not marked synthetic#354
Philippus wants to merge 1 commit intospray:release/1.3.xfrom
Philippus:remove-synthetic-workaround

Conversation

@Philippus
Copy link
Copy Markdown

scala/scala3#14083 was fixed in Scala 3.1.2.

Copy link
Copy Markdown
Contributor

@raboof raboof left a comment

Choose a reason for hiding this comment

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

hmm, even if we ourselves update to 3.1.2 wouldn't this still fail on classes that were compiled with 3.1.0?

@jrudolph
Copy link
Copy Markdown
Member

jrudolph commented May 9, 2025

hmm, even if we ourselves update to 3.1.2 wouldn't this still fail on classes that were compiled with 3.1.0?

Agreed probably not worth fixing for 3.1.x.

@Philippus
Copy link
Copy Markdown
Author

hmm, even if we ourselves update to 3.1.2 wouldn't this still fail on classes that were compiled with 3.1.0?

Agreed probably not worth fixing for 3.1.x.

It can be part of this PR then: #342.

@Philippus Philippus closed this Aug 26, 2025
@Philippus Philippus deleted the remove-synthetic-workaround branch August 26, 2025 12:12
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