test(reader): drop redundant Math.max alloc guards in TestArrays

allocate(0, n) is legal and tested (empty bytes/bools cases), so the
Math.max(1, ...) floors were dead code that made bytes()/bools() diverge
from the other builders. Remove for consistency.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: dfa1

reader/src/test/java/io/github/dfa1/vortex/reader/array/TestArrays.java

@@ -83,7 +83,7 @@ public static ShortArray shorts(short... vs) {
     /// @param vs element values
     /// @return read-only materialized byte array
     public static ByteArray bytes(byte... vs) {
-        MemorySegment seg = Arena.ofAuto().allocate(Math.max(1, vs.length), 1);
+        MemorySegment seg = Arena.ofAuto().allocate(vs.length, 1);
         for (int i = 0; i < vs.length; i++) {
             seg.set(ValueLayout.JAVA_BYTE, i, vs[i]);
         }
@@ -95,7 +95,7 @@ public static ByteArray bytes(byte... vs) {
     /// @param vs element values
     /// @return read-only materialized bool array
     public static BoolArray bools(boolean... vs) {
-        MemorySegment seg = Arena.ofAuto().allocate(Math.max(1, (vs.length + 7) / 8), 1);
+        MemorySegment seg = Arena.ofAuto().allocate((vs.length + 7) / 8, 1);
         for (int i = 0; i < vs.length; i++) {
             if (vs[i]) {
                 long byteIdx = i >>> 3;