Special handling for primitive arrays

src/main/kotlin/io/exoquery/pprint/Walker.kt

@@ -56,6 +56,10 @@ interface Walker {
 
   fun treeify(x: Any?, escapeUnicode: Boolean, showFieldNames: Boolean): Tree {
     fun treeifySame(x: Any?) = treeify(x, escapeUnicode, showFieldNames)
+
+    fun <T> applyArray(name: String, seq: Sequence<T>) =
+      Tree.Apply(name, seq.map {x -> treeifySame(x)}.iterator())
+
     return when {
 
       x == null -> Tree.Literal("null")
@@ -133,9 +137,18 @@ interface Walker {
           Tree.Literal("non-empty iterator")
       }
 
-      x is Array<*> -> Tree.Apply("Array", x.asSequence().map {x -> treeify(x, escapeUnicode, showFieldNames)}.iterator())
+      x is BooleanArray -> applyArray("BooleanArray", x.asSequence())
+      x is ByteArray -> applyArray("ByteArray", x.asSequence())
+      x is CharArray -> applyArray("CharArray", x.asSequence())
+      x is DoubleArray -> applyArray("DoubleArray", x.asSequence())
+      x is FloatArray -> applyArray("FloatArray", x.asSequence())
+      x is IntArray -> applyArray("IntArray", x.asSequence())
+      x is LongArray -> applyArray("LongArray", x.asSequence())
+      x is ShortArray -> applyArray("ShortArray", x.asSequence())
+
+      x is Array<*> -> applyArray("Array", x.asSequence())
 
-      x is Sequence<*> -> Tree.Apply("Sequence", x.map {x -> treeify(x, escapeUnicode, showFieldNames)}.iterator())
+      x is Sequence<*> -> Tree.Apply("Sequence", x.map {x -> treeifySame(x)}.iterator())
 
       x is Pair<*, *> -> {
         // We could also do Tree.Infix(treeifySame(x.first), "to", treeifySame(x.second)), so it would be "a to b" not sure if that is better or worse

src/test/kotlin/io/exoquery/pprint/HorizontalTests.kt

@@ -109,6 +109,16 @@ class HorizontalTests : FunSpec({
       test("Iterator") { Check(listOf('1', '2', '3'), "List('1', '2', '3')") }
 
       test("Array") { Check(arrayOf(1, 2, 3), "Array(1, 2, 3)") }
+
+      test("BooleanArray") { Check(booleanArrayOf(true, false, true), "BooleanArray(true, false, true)") }
+      test("ByteArray") { Check(byteArrayOf(1, 2, 3), "ByteArray(1, 2, 3)") }
+      test("CharArray") { Check(charArrayOf('a', 'b', 'c'), "CharArray('a', 'b', 'c')") }
+      test("DoubleArray") { Check(doubleArrayOf(1.0, 2.0, 3.0), "DoubleArray(1.0, 2.0, 3.0)") }
+      test("FloatArray") { Check(floatArrayOf(1.0F, 2.0F, 3.0F), "FloatArray(1.0F, 2.0F, 3.0F)") }
+      test("IntArray") { Check(intArrayOf(1, 2, 3), "IntArray(1, 2, 3)") }
+      test("LongArray") { Check(longArrayOf(1L, 2L, 3L), "LongArray(1L, 2L, 3L)") }
+      test("ShortArray") { Check(shortArrayOf(1, 2, 3), "ShortArray(1, 2, 3)") }
+
       test("Seq") { Check(listOf(1, 2, 3), "List(1, 2, 3)") }
       test("List") { Check(listOf("1", "2", "3"), """List("1", "2", "3")""") }
       //test("Vector") { Check(Vector('omg, 'wtf, 'bbq), """Vector('omg, 'wtf, 'bbq)""") }