Skip to content

Backport "Make sure definition tree has the defined symbol" to 3.3 LTS #81

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

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

tgodzik
Copy link

@tgodzik tgodzik commented Feb 17, 2025

Backports scala#21851 to the 3.3.6.

PR submitted by the release tooling.
[skip ci]

odersky and others added 2 commits February 17, 2025 16:15
It turns out it could have the wrong symbol referring to a same-named definition
in the superclass under some recursive definition of a self type. This caused a crash
in pickler in scala#21755 because we now have two different definitions in two different classes that
have the same symbol.

Fixes scala#21755
It turns out it could have the wrong symbol referring to a same-named definition
in the superclass under some recursive definition of a self type. This caused a crash
in pickler in scala#21755 because we now have two different definitions in two different classes that
have the same symbol.

Fixes scala#21755

[Cherry-picked 03ee583][modified]
Base automatically changed from backport-lts-3.3-21677 to lts-3.3 February 19, 2025 17:09
@tgodzik
Copy link
Author

tgodzik commented Feb 19, 2025

No regressions detected in the community build up to backport-lts-3.3-21466.

Reference

@tgodzik tgodzik merged commit 6abffe5 into lts-3.3 Feb 19, 2025
22 checks passed
@tgodzik tgodzik deleted the backport-lts-3.3-21851 branch February 19, 2025 17:09
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