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.

Sign up for GitHub

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

Jump to bottom

Change order of proxy evaluation when inlining #13755

Merged
merged 1 commit into from Oct 20, 2021
Merged

Change order of proxy evaluation when inlining #13755

merged 1 commit into from Oct 20, 2021

Conversation

odersky
Copy link
Contributor

@odersky odersky commented Oct 15, 2021

Fixes #13747

@soronpo
Copy link
Contributor

soronpo commented Oct 15, 2021 •
edited

I can confirm that this fixes a more complex example in my codebase before the minimization. Thanks!

@nicolasstucki nicolasstucki merged commit faf9538 into scala:master Oct 20, 2021
@nicolasstucki nicolasstucki deleted the fix-13747 branch October 20, 2021 10:32
@Kordyjan Kordyjan added this to the 3.1.2 milestone Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nicolasstucki nicolasstucki nicolasstucki approved these changes

Assignees

@nicolasstucki nicolasstucki

Labels
None yet
Projects
None yet
Milestone
3.1.2
Development

Successfully merging this pull request may close these issues.

Inlining affects order of execution
4 participants
@odersky @soronpo @nicolasstucki @Kordyjan