-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Failure to load Seq with Scala 2 library TASTy with CC annotations #19586
Labels
area:experimental:cc
Capture checking related
cc-experiment
Intended to be merged with cc-experiment branch on origin
itype:bug
Comments
nicolasstucki
added
itype:bug
cc-experiment
Intended to be merged with cc-experiment branch on origin
labels
Feb 1, 2024
It happens in the |
#19603 fixes the basics and we can now compile most tests. I tried to go further with
Here's the current list of failures:
We need to follow up on this next. |
odersky
added a commit
that referenced
this issue
Feb 4, 2024
Two measures: - Introduce a new annotation @retainsCap that's equivalent to `@retains(cap)` but that does not need to load Seq to analyze a vararg argument. - Don't use `&` when computing self types of classes with explicitly declared capturing self types. Partial fix of #19586
nicolasstucki
added a commit
to dotty-staging/dotty
that referenced
this issue
Feb 5, 2024
Tests the fixes in scala#19603. Follow-up of scala#19586.
odersky
added a commit
that referenced
this issue
Feb 5, 2024
Moving the followup discussion to #19652. The original issue is fixed and we have regression tests. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
area:experimental:cc
Capture checking related
cc-experiment
Intended to be merged with cc-experiment branch on origin
itype:bug
Followup to #19398.
Minimized code
Output
The text was updated successfully, but these errors were encountered: