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

Backport "Revert "Drop redundant butNot = Param clause in isAnchor"" to LTS #22127

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

WojciechMazur
Copy link
Contributor

Backports #21566 to the 3.3.5.

PR submitted by the release tooling.
[skip ci]

This reverts commit 9d88c80.
Closes #21521

The `ClassTypeParamCreationFlags` include both `TypeParam` and `Deferred`.
In effect, a class type parameter was incorrectly considered as an anchor.

For a failing example, one can try asserting:
```scala
|| sym.is(Deferred).ensuring(_ == sym.is(Deferred, butNot = Param))
```
in `ImplicitRunInfo#isAnchor` and a test with `summon[Ordering[Int]]`.

In that example, at least,
the flags happen to be set by `Scala2Unpickler#readDisambiguatedSymbol`
src/dotty/tools/dotc/core/unpickleScala2/Scala2Unpickler.scala:562.

[Cherry-picked 1b1dd16][modified]
Base automatically changed from backport-lts-3.3-20318 to lts-3.3 December 4, 2024 17:21
@WojciechMazur
Copy link
Contributor Author

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

Reference

@WojciechMazur WojciechMazur merged commit 2d50b3b into lts-3.3 Dec 4, 2024
19 checks passed
@WojciechMazur WojciechMazur deleted the backport-lts-3.3-21566 branch December 4, 2024 17:21
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.

1 participant