Skip to content

Conversation

@odersky
Copy link
Contributor

@odersky odersky commented Dec 12, 2014

Fixed problems with methods getting too large a scope in LambdaLift. Review by @DarkDimius

Was: erasure of underlying type.
Now: erased typeRef of underlying class.

The differ if there is a self type. In this case,
a this type did erase to teh erasure of the self type, which
is wrong.
A lot of cases were missing before and caused failures
in the newly added test in TreeChecker#typedThis.

Now we assure that all this references appear inside
the referenced class.
DarkDimius added a commit that referenced this pull request Dec 15, 2014
@DarkDimius DarkDimius merged commit 8088ef2 into scala:master Dec 15, 2014
@allanrenucci allanrenucci deleted the fix/#266-lambdaLift branch December 14, 2017 19:21
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Apr 29, 2025
Backport "Check exhaustivity of any case class" to 3.3 LTS
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 17, 2025
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 17, 2025
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 18, 2025
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 18, 2025
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 18, 2025
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 18, 2025
tgodzik added a commit that referenced this pull request Sep 19, 2025
…23943)

This reverts commit 91aacdb, reversing
changes made to cc130d4.

Unfortunately, this caused repositories such as pekko to have some
unexpected warnings.
tgodzik added a commit to tgodzik/scala3 that referenced this pull request Sep 26, 2025
Revert "Merge pull request scala#275 from scala/backport-lts-3.3-22604"
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