Merge pull request #2225 from dotty-staging/fix-#2192

Fix #2212: Avoid imports in the wrong namespace

2 files changed, 21 insertions, 3 deletions

diff --git a/tests/pos/i2212.scala b/tests/pos/i2212.scala
new file mode 100644
index 000000000..416c8ca04
--- /dev/null
+++ b/tests/pos/i2212.scala
@@ -0,0 +1,19 @@
+package object squants {
+  type Time = squants.time.Time
+}
+package squants.time {
+  class Time
+  object Time { def x = 2 }
+}
+package squants.velocity {
+  import squants.time._  // <-- imports `Time` value
+  import squants.Time // <-- imports  type alias
+  object Velocity { Time.x }
+}
+
+import scala.math.BigDecimal.RoundingMode
+import scala.math.BigDecimal.RoundingMode.RoundingMode
+
+object Money {
+  def foo(round: RoundingMode = RoundingMode.HALF_EVEN): Int = ???
+}
diff --git a/tests/run/t5857.scala b/tests/run/t5857.scala
index eabf5dc86..fe67a7546 100644
--- a/tests/run/t5857.scala
+++ b/tests/run/t5857.scala
@@ -36,10 +36,9 @@ object Test {
       b
     }
 
-    // whatever it is, it should be less than, say, 250ms
+    // whatever it is, it should be less than, say, 1000ms
     // if `max` involves traversal, it takes over 5 seconds on a 3.2GHz i7 CPU
     //println(exectime)
-    assert(exectime < 250, exectime)
+    assert(exectime < 1000, exectime)
   }
-
 }