writing Java in vi isn't always easy

rokob · rokob · commit b4ff1bba5c6f · 2018-12-03T12:00:45.000-08:00
diff --git a/rollbar-java/src/main/java/com/rollbar/notifier/config/ConfigBuilder.java b/rollbar-java/src/main/java/com/rollbar/notifier/config/ConfigBuilder.java
@@ -501,7 +501,11 @@ private static class ConfigImpl implements Config {
       this.uuidGenerator = builder.uuidGenerator;
       this.sender = builder.sender;
       this.proxy = builder.proxy;
-      this.appPackages = builder.appPackages != null ? builder.appPackages : Collections.emptyList();
+      if (builder.appPackages == null) {
+        this.appPackages = Collections.<String>emptyList();
+      } else {
+        this.appPackages = builder.appPackages;
+      }
       this.handleUncaughtErrors = builder.handleUncaughtErrors;
       this.enabled = builder.enabled;
     }
diff --git a/rollbar-java/src/main/java/com/rollbar/notifier/util/BodyFactory.java b/rollbar-java/src/main/java/com/rollbar/notifier/util/BodyFactory.java
@@ -12,6 +12,7 @@
 import com.rollbar.notifier.wrapper.ThrowableWrapper;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Map;
 
 /**
  * Body factory helper to build the proper body depending on the throwable or the description.
@@ -81,7 +82,7 @@ private static TraceChain traceChain(ThrowableWrapper throwableWrapper, String d
 
   private static List<Frame> frames(ThrowableWrapper throwableWrapper) {
     StackTraceElement[] elements = throwableWrapper.getStackTrace();
-    CacheFrame[] cachedFrames = ThrowableCache.get(throwableWrapper.throwable());
+    CacheFrame[] cachedFrames = ThrowableCache.get(throwableWrapper.getThrowable());
     int j = 0;
     if (cachedFrames != null) {
       j = cachedFrames.length - 1;
