Merge pull request scala#2611 from retronym/ticket/6481

SI-6841 Fix bug at the intersection of DelayedInit and named args

Author: paulp

src/compiler/scala/tools/nsc/transform/Constructors.scala

@@ -523,7 +523,10 @@ abstract class Constructors extends Transform with ast.TreeDSL {
 
       /** Return a pair consisting of (all statements up to and including superclass and trait constr calls, rest) */
       def splitAtSuper(stats: List[Tree]) = {
-        def isConstr(tree: Tree) = (tree.symbol ne null) && tree.symbol.isConstructor
+        def isConstr(tree: Tree): Boolean = tree match {
+          case Block(_, expr) => isConstr(expr)  // SI-6481 account for named argument blocks
+          case _              => (tree.symbol ne null) && tree.symbol.isConstructor
+        }
         val (pre, rest0) = stats span (!isConstr(_))
         val (supercalls, rest) = rest0 span (isConstr(_))
         (pre ::: supercalls, rest)

test/files/run/t6481.check

@@ -0,0 +1,4 @@
+delayed init
+new foo(1, 2)
+delayed init
+new foo(b = 2, a = 1)

test/files/run/t6481.scala

@@ -0,0 +1,13 @@
+abstract class foo(a: Int, b: Int) extends scala.DelayedInit {
+  def delayedInit(x: => Unit) {
+    println("delayed init");
+    x
+  }
+}
+
+object Test {
+  def main(args: Array[String]) {
+    new foo(1, 2) { println("new foo(1, 2)") }
+    new foo(b = 2, a = 1) { println("new foo(b = 2, a = 1)") }
+  }
+}