diff options
| author |  lpino <lpino@google.com> | 2018-05-17 05:00:32 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-05-17 05:02:17 -0700 |
| commit | e1ada6d351cb714c9e5694e1e58cd2b761c84784 (patch) | |
| tree | b2b1f3813a551d05afe9d8dafe30dc3d04d54682 /src/main/java/com/google/devtools/build/lib/buildeventservice | |
| parent | 5724682dd7b2134c837522a930fd6844d45a281c (diff) | |
Remove AbruptException(ExitCode exitCode) constructor.
This constructor was creating an Exception with a null message leading to possible
NullPointerExceptions in a few places in our codebase. The call sites have
been replaced with calls to AbruptException(String message, ExitCode exitCode) with
a meaningful message.
PiperOrigin-RevId: 196973540
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/buildeventservice')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/buildeventservice/BuildEventServiceTransport.java | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/buildeventservice/BuildEventServiceTransport.java b/src/main/java/com/google/devtools/build/lib/buildeventservice/BuildEventServiceTransport.java index 21272938b4..a3cafc2bf6 100644 --- a/src/main/java/com/google/devtools/build/lib/buildeventservice/BuildEventServiceTransport.java +++ b/src/main/java/com/google/devtools/build/lib/buildeventservice/BuildEventServiceTransport.java @@ -341,7 +341,8 @@ public class BuildEventServiceTransport implements BuildEventTransport { String message = errorMessageFromException(t); report(ERROR, message); - moduleEnvironment.exit(new AbruptExitException(ExitCode.PUBLISH_ERROR)); + moduleEnvironment.exit(new AbruptExitException( + "BuildEventServiceTransport internal error", ExitCode.PUBLISH_ERROR)); } private void maybeReportUploadError() { |