Skip to content

Commit 495b07e

Browse files
committed
Fix #651 Generalize criterion for isOpAssign
Methods like + can have multiple parameters. In that case += also takes multiple parameters.
1 parent ae8a246 commit 495b07e

File tree

3 files changed

+3
-2
lines changed

3 files changed

+3
-2
lines changed

src/dotty/tools/dotc/ast/TreeInfo.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ trait TreeInfo[T >: Untyped <: Type] { self: Trees.Instance[T] =>
3333
}
3434

3535
def isOpAssign(tree: Tree) = unsplice(tree) match {
36-
case Apply(fn, _ :: Nil) =>
36+
case Apply(fn, _ :: _) =>
3737
unsplice(fn) match {
3838
case Select(_, name) if name.isOpAssignmentName => true
3939
case _ => false
File renamed without changes.

tests/pending/run/colltest1.scala renamed to tests/run/colltest1.scala

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,8 @@ object Test extends dotty.runtime.LegacyApp {
139139

140140
def setTest(empty: => Set[String]): Unit = {
141141
var s = empty + "A" + "B" + "C"
142-
s += ("D", "E", "F")
142+
s += "D"
143+
s = s + ("D", "E", "F")
143144
s ++= List("G", "H", "I")
144145
s ++= ('J' to 'Z') map (_.toString)
145146
assert(s forall (s contains))

0 commit comments

Comments
 (0)