Skip to content

Ycheck failure after frontend and MegaPhase{firstTransform, checkReentrant, elimJavaPackages} #3669

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
allanrenucci opened this issue Dec 15, 2017 · 0 comments
Closed

Ycheck failure after frontend and MegaPhase{firstTransform, checkReentrant, elimJavaPackages} #3669

allanrenucci opened this issue Dec 15, 2017 · 0 comments
Labels
exp:advanced itype:bug prio:low

Comments

@allanrenucci
Copy link
Contributor 

import java.nio.file._

class Test {
  def test = {
    val p = Paths.get("")
  }
}
dotc Test.scala -Ycheck:frontend
dotc Test.scala -Ycheck:elimJavaPackages

Both fail
allanrenucci added a commit to dotty-staging/dotty that referenced this issue Dec 16, 2017
@allanrenucci
Fix scala#3669: Treat java repeated arguments like scalac's
cf158ba 
Java repeated arguments are now treated as scala repeated arguments. They
are now replaced by a java array during ElimRepeated instead of Typer.

This change let us Ycheck one more MegaPhase. Previously java repeated
arguments would be replaced by a java array during typechecking which
prevented us from retypechecking the tree: Array[T] found, expected T*
allanrenucci added a commit that referenced this issue Dec 18, 2017
@allanrenucci
Merge pull request #3674 from dotty-staging/ycheck
f4a22a8 
Fix #3669: Treat java repeated arguments like scalac's
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
exp:advanced itype:bug prio:low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@allanrenucci