Fix null handling

Took 5 minutes
Darkyenus · Jan 30, 2017 · 673803b · 673803b
1 parent d1b2ce6
commit 673803b
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/build.sbt b/build.sbt
@@ -1,7 +1,7 @@
 
 name := "tproll"
 
-version := "1.2"
+version := "1.2.1"
 
 organization := "com.darkyen"
 

diff --git a/src/main/java/com/darkyen/tproll/util/PrettyPrinter.java b/src/main/java/com/darkyen/tproll/util/PrettyPrinter.java
@@ -25,7 +25,9 @@ public static void append(StringBuilder sb, Object item){
 
     public static void append(StringBuilder sb, Object item, int maxArrayElements){
         //To use faster overloads
-        if (item instanceof Boolean) {
+        if (item == null) {
+            sb.append((String)null);
+        } else if (item instanceof Boolean) {
             sb.append((boolean)item);
         } else if (item instanceof Character) {
             sb.append((char)item);

diff --git a/src/test/java/unit/LoggerTest.java b/src/test/java/unit/LoggerTest.java
@@ -134,6 +134,12 @@ public void fullArray() {
         assertLogIs(INFO_PREFIX + "I have these banknotes: String\\[100, 200, -560, 1M Zimbabwe \\$\\]");
     }
 
+    @Test
+    public void nullArg() {
+        log.info("Reason: {}", (Object)null);
+        assertLogIs(INFO_PREFIX + "Reason: null");
+    }
+
     @SuppressWarnings("serial")
     private static final class DummyException extends Exception {
         public DummyException(String message) {