Delegate the default implementation of `BugReporter#sendBugReport(Thr…

…owable)` to `BugReporter#sendBugReport(Throwable, List, String...)`. PiperOrigin-RevId: 364682636
bazelbuild · Mar 24, 2021 · a8ef70e · a8ef70e
1 parent db9467f
commit a8ef70e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/bugreport/BugReport.java b/src/main/java/com/google/devtools/build/lib/bugreport/BugReport.java
@@ -276,11 +276,6 @@ private static void logException(Throwable exception, List<String> args, String.
 
   private static final class DefaultBugReporter implements BugReporter {
 
-    @Override
-    public void sendBugReport(Throwable exception) {
-      BugReport.sendBugReport(exception);
-    }
-
     @Override
     public void sendBugReport(Throwable exception, List<String> args, String... values) {
       BugReport.sendBugReport(exception, args, values);

diff --git a/src/main/java/com/google/devtools/build/lib/bugreport/BugReporter.java b/src/main/java/com/google/devtools/build/lib/bugreport/BugReporter.java
@@ -13,6 +13,7 @@
 // limitations under the License.
 package com.google.devtools.build.lib.bugreport;
 
+import com.google.common.collect.ImmutableList;
 import java.util.List;
 
 /**
@@ -29,9 +30,11 @@ static BugReporter defaultInstance() {
   }
 
   /** Reports an exception, see {@link BugReport#sendBugReport(Throwable)}. */
-  void sendBugReport(Throwable exception);
+  default void sendBugReport(Throwable exception) {
+    sendBugReport(exception, /*args=*/ ImmutableList.of());
+  }
 
-  /** Reports an exception, see {@link BugReport#sendBugReport(Throwable, List, String[])}. */
+  /** Reports an exception, see {@link BugReport#sendBugReport(Throwable, List, String...)}. */
   void sendBugReport(Throwable exception, List<String> args, String... values);
 
   /** See {@link BugReport#handleCrash}. */