Do not crash when encountering double semi-colon after package statement

MOE_MIGRATED_REVID=240588095
google · Apr 3, 2019 · 5e114ca · 5e114ca
1 parent 9b0f032
commit 5e114ca
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/core/src/main/java/com/google/googlejavaformat/java/JavaInputAstVisitor.java b/core/src/main/java/com/google/googlejavaformat/java/JavaInputAstVisitor.java
@@ -347,6 +347,7 @@ public Void visitCompilationUnit(CompilationUnitTree node, Void unused) {
       builder.forcedBreak();
       first = false;
     }
+    dropEmptyDeclarations();
     if (!node.getImports().isEmpty()) {
       if (!first) {
         builder.blankLineWanted(BlankLineWanted.YES);

diff --git a/core/src/test/resources/com/google/googlejavaformat/java/testdata/CL240367479.input b/core/src/test/resources/com/google/googlejavaformat/java/testdata/CL240367479.input
@@ -0,0 +1,6 @@
+package foo;;
+
+import com.google.second.Foo;
+import com.google.first.Bar;
+
+public class Blim {}
diff --git a/core/src/test/resources/com/google/googlejavaformat/java/testdata/CL240367479.output b/core/src/test/resources/com/google/googlejavaformat/java/testdata/CL240367479.output
@@ -0,0 +1,6 @@
+package foo;
+;
+import com.google.second.Foo;
+import com.google.first.Bar;
+
+public class Blim {}