Clean up BugReport comments And an unnecessary local variable. RELNOTES: None. PiperOrigin-RevId: 314468222
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 index 8a6f37d..badb543 100644 --- a/src/main/java/com/google/devtools/build/lib/bugreport/BugReport.java +++ b/src/main/java/com/google/devtools/build/lib/bugreport/BugReport.java
@@ -142,14 +142,13 @@ } /** - * Print, log, send a bug report, and then cause the current Blaze command to fail with the - * specified exit code, and then cause the jvm to terminate. + * Print, log, and then cause the current Blaze command to fail with the specified exit code, and + * then cause the jvm to terminate. * * <p>Has no effect if another crash has already been handled by {@link BugReport}. */ public static void handleCrashWithoutSendingBugReport( Throwable throwable, ExitCode exitCode, String... args) { - // TODO(b/78555988): allow call sites to specialize DetailedExitCode handleCrash( throwable, /*sendBugReport=*/ false, @@ -164,7 +163,6 @@ * <p>Has no effect if another crash has already been handled by {@link BugReport}. */ public static void handleCrash(Throwable throwable, ExitCode exitCode, String... args) { - // TODO(b/78555988): allow call sites to specialize DetailedExitCode handleCrash( throwable, /*sendBugReport=*/ true, @@ -191,8 +189,7 @@ boolean sendBugReport, DetailedExitCode detailedExitCode, String... args) { - ExitCode exitCodeToUse = detailedExitCode.getExitCode(); - int numericExitCode = exitCodeToUse.getNumericExitCode(); + int numericExitCode = detailedExitCode.getExitCode().getNumericExitCode(); try { synchronized (LOCK) { if (TestType.isInTest()) {