Skip to content

EtaExpansion.liftArgs removes NamedArgs #1096

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
Closed
DarkDimius opened this issue Feb 17, 2016 · 0 comments
Closed

EtaExpansion.liftArgs removes NamedArgs #1096

DarkDimius opened this issue Feb 17, 2016 · 0 comments
Assignees
@odersky
Labels
stat:fix available

Comments

@DarkDimius
Copy link
Contributor

This could make correlating trees with positions harder.
We have an agreement that NamedArgs should survive until FirstTransform.

The place where it's eliminated:
https://github.com/lampepfl/dotty/blob/master/src/dotty/tools/dotc/typer/EtaExpansion.scala#L53
odersky added a commit to dotty-staging/dotty that referenced this issue Feb 18, 2016
@odersky
Keep named arguments during eta expansion
cdfbdb1 
Fixes scala#1096.
@odersky odersky mentioned this issue Feb 18, 2016
Merged
odersky referenced this issue Feb 19, 2016
@odersky
Merge pull request #1102 from dotty-staging/fix-#1096
07bf063 
Keep named arguments during eta expansion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@odersky odersky

Labels
stat:fix available
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@odersky @DarkDimius