Merge pull request #12516 from dotty-staging/fix-12249

OlivierBlanvillain · web-flow · commit d26549d8e167 · 2021-05-19T17:30:29.000+02:00
Fix #12249: don't export java defined modules
diff --git a/compiler/src/dotty/tools/dotc/typer/Namer.scala b/compiler/src/dotty/tools/dotc/typer/Namer.scala
@@ -1000,6 +1000,8 @@ class Namer { typer: Typer =>
           ) match
               case Some(other) => No(i"overrides ${other.showLocated}, which is already a member of $cls")
               case None => Yes
+        else if sym.isAllOf(JavaModule) then
+          Skip
         else Yes
       }
 
diff --git a/tests/neg/12249.scala b/tests/neg/12249.scala
@@ -0,0 +1,9 @@
+
+export java.util.UUID
+
+def bar = println(new UUID(1, 2)) // OK
+
+def foo = println(UUID.randomUUID()) // error
+//                ^^^^
+//                Not found: UUID
+

Original file line number	Diff line number	Diff line change
`@@ -1000,6 +1000,8 @@ class Namer { typer: Typer =>`
`1000`	`1000`	`) match`
`1001`	`1001`	`case Some(other) => No(i"overrides ${other.showLocated}, which is already a member of $cls")`
`1002`	`1002`	`case None => Yes`
	`1003`	`+ else if sym.isAllOf(JavaModule) then`
	`1004`	`+ Skip`
`1003`	`1005`	`else Yes`
`1004`	`1006`	`}`
`1005`	`1007`