Merge pull request #13843 from adpi2/fix-print-summary

smarter · web-flow · commit 49b6c08098a6 · 2021-11-01T18:49:49.000+01:00
Fix error summary printed twice
diff --git a/sbt-bridge/src/dotty/tools/xsbt/CompilerBridgeDriver.java b/sbt-bridge/src/dotty/tools/xsbt/CompilerBridgeDriver.java
@@ -91,10 +91,10 @@ public int compare(VirtualFile x0, VirtualFile x1) {
           callback.problem(problem.category(), problem.position(), problem.message(), problem.severity(),
             true);
         }
+      } else {  
+        delegate.printSummary();
       }
 
-      delegate.printSummary();
-
       if (delegate.hasErrors()) {
         log.debug(() -> "Compilation failed");
         throw new InterfaceCompileFailed(args, delegate.problems(), "Compilation failed");

Original file line number	Diff line number	Diff line change
`@@ -91,10 +91,10 @@ public int compare(VirtualFile x0, VirtualFile x1) {`
`91`	`91`	`callback.problem(problem.category(), problem.position(), problem.message(), problem.severity(),`
`92`	`92`	`true);`
`93`	`93`	`}`
	`94`	`+ } else {`
	`95`	`+ delegate.printSummary();`
`94`	`96`	`}`
`95`	`97`
`96`		`- delegate.printSummary();`
`97`		`-`
`98`	`98`	`if (delegate.hasErrors()) {`
`99`	`99`	`log.debug(() -> "Compilation failed");`
`100`	`100`	`throw new InterfaceCompileFailed(args, delegate.problems(), "Compilation failed");`