Apply suggestions from code review

odersky · nicolasstucki · web-flow · commit 46d232467fd4 · 2019-08-06T21:53:50.000+02:00
Co-Authored-By: Nicolas Stucki &lt;nicolas.stucki@gmail.com&gt;
diff --git a/compiler/src/dotty/tools/dotc/parsing/Parsers.scala b/compiler/src/dotty/tools/dotc/parsing/Parsers.scala
@@ -1332,7 +1332,7 @@ object Parsers {
       case DO =>
         in.errorOrMigrationWarning(
           i"""`do <body> while <cond>' is no longer supported,
-             |use `while ({<body> ; <cond>}) ()' instead.
+             |use `while {<body> ; <cond>} do ()' instead.
              |The statement can be rewritten automatically under -language:Scala2 -migration -rewrite.
            """)
         val start = in.skipToken()
@@ -1353,7 +1353,7 @@ object Parsers {
             }
             patch(source, cond.span.endPos, "}) ()")
           }
-          WhileDo(Block(Block(Nil, body), Block(Nil, cond)), Literal(Constant(())))
+          WhileDo(Block(body :: Nil, cond), Literal(Constant(())))
         }
       case TRY =>
         val tryOffset = in.offset

Original file line number	Diff line number	Diff line change
`@@ -1332,7 +1332,7 @@ object Parsers {`
`1332`	`1332`	`case DO =>`
`1333`	`1333`	`in.errorOrMigrationWarning(`
`1334`	`1334`	i"""`do <body> while <cond>' is no longer supported,
`1335`		- \|use `while ({<body> ; <cond>}) ()' instead.
	`1335`	+ \|use `while {<body> ; <cond>} do ()' instead.
`1336`	`1336`	`\|The statement can be rewritten automatically under -language:Scala2 -migration -rewrite.`
`1337`	`1337`	`""")`
`1338`	`1338`	`val start = in.skipToken()`
`@@ -1353,7 +1353,7 @@ object Parsers {`
`1353`	`1353`	`}`
`1354`	`1354`	`patch(source, cond.span.endPos, "}) ()")`
`1355`	`1355`	`}`
`1356`		`- WhileDo(Block(Block(Nil, body), Block(Nil, cond)), Literal(Constant(())))`
	`1356`	`+ WhileDo(Block(body :: Nil, cond), Literal(Constant(())))`
`1357`	`1357`	`}`
`1358`	`1358`	`case TRY =>`
`1359`	`1359`	`val tryOffset = in.offset`