Skip to content

Conversation

tgodzik
Copy link
Collaborator

@tgodzik tgodzik commented Sep 23, 2024

Fixes #3973

@tgodzik tgodzik force-pushed the support-2.13.15 branch 3 times, most recently from af8f720 to d32245f Compare September 23, 2024 14:00
@@ -42,6 +42,6 @@ class Synthetic/*<=example.Synthetic#*/ {
a/*<=local11*/ <- scala.concurrent.Future/*=>scala.concurrent.Future.*/.successful/*=>scala.concurrent.Future.successful().*/(1)
b/*<=local12*/ <- scala.concurrent.Future/*=>scala.concurrent.Future.*/.successful/*=>scala.concurrent.Future.successful().*/(2)
if a/*=>local11*/ </*=>scala.Int#`<`(+3).*/ b/*=>local12*/
} yield a/*=>local11*/
} yield a/*=>local13*/
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@tgodzik tgodzik marked this pull request as ready for review September 23, 2024 14:36
@tgodzik tgodzik requested a review from kitbellew September 23, 2024 14:37
Copy link
Collaborator

@kitbellew kitbellew left a comment

Choose a reason for hiding this comment

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

i am assuming you understand the changes, so asked anyway...

if (ScalaVersion.atLeast212_14) body.expected
else
// Predef.type etc. was fixed in 2.12.14
body.expected.replace("Predef/*=>scala.Predef.*/.type", "Predef.type")
Copy link
Collaborator

Choose a reason for hiding this comment

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

is this unrelated? cleanup for 2.12?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yep, that's not necessary anymore and I was already cleaning this up.

@@ -1729,11 +1729,6 @@ Synthetics:
a,
b
) == (1, 2))}).flatMap[Tuple6[Int, Int, Int, Int, Tuple4[Int, Int, Int, Int], Tuple4[Int, Int, Int, Int]]]({(b) => orig(List((a, b))).withFilter({(check$ifrefutable$1) => orig()}).withFilter({(local7) => orig((
a,
Copy link
Collaborator

Choose a reason for hiding this comment

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

why did this disappear?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I would assume that some internal desugarings changed, but I honestly think these synthetics are not being used currently by anyone.

@kitbellew kitbellew merged commit 2d284c2 into scalameta:main Sep 23, 2024
24 checks passed
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.

Please publish for Scala 2.13.15
2 participants