From c167d1e9dc03aebdc7dd8f59cbde4e4d783fee0e Mon Sep 17 00:00:00 2001 From: Liam Miller-Cushon Date: Thu, 17 Nov 2016 00:41:28 +0000 Subject: Fix a javac-turbine tree pruning bug When rewriting blocks to `{ throws new AssertionError(); }`, use a fully-qualified name for `java.lang.AssertionError` in case we're inside the declaration of a type with the same simple name. Also, only add the throws expression statement when rewriting lambda body blocks, since that's the only place it matters. -- MOS_MIGRATED_REVID=139397324 --- .../com/google/devtools/build/java/turbine/javac/TreePrunerTest.java | 2 -- 1 file changed, 2 deletions(-) (limited to 'src/java_tools/buildjar/javatests') diff --git a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/TreePrunerTest.java b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/TreePrunerTest.java index bebb093e14..ac383a43ba 100644 --- a/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/TreePrunerTest.java +++ b/src/java_tools/buildjar/javatests/com/google/devtools/build/java/turbine/javac/TreePrunerTest.java @@ -106,10 +106,8 @@ public class TreePrunerTest { String[] expected = { "class Test {", // " {", - " throw new AssertionError();", " }", " static {", - " throw new AssertionError();", " }", "}", }; -- cgit v1.2.3