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()) {