Merge pull request #257 from apache/better_test_coverage

More comprehensive tests.

Author: leerho

src/main/java/org/apache/datasketches/memory/internal/NativeWritableBufferImpl.java

@@ -41,10 +41,10 @@ final class NativeWritableBufferImpl extends WritableBufferImpl {
 
   //Pass-through ctor
   NativeWritableBufferImpl(
-      final Arena arena,
       final MemorySegment seg,
       final int typeId,
-      final MemoryRequestServer memReqSvr) {
+      final MemoryRequestServer memReqSvr,
+      final Arena arena) {
     super(seg, typeId, memReqSvr, arena);
   }
 

src/main/java/org/apache/datasketches/memory/internal/ResourceImpl.java

@@ -220,7 +220,7 @@ static final WritableBuffer selectBuffer(
     final MemoryRequestServer memReqSvr2 = (byteBufferType || mapType) ? null : memReqSvr;
     final WritableBuffer wbuf;
     if (nativeBOType) {
-      wbuf = new NativeWritableBufferImpl(null, segment, type, memReqSvr2);
+      wbuf = new NativeWritableBufferImpl(segment, type, memReqSvr2, null);
     } else { //non-native BO
       wbuf = new NonNativeWritableBufferImpl(segment, type, memReqSvr2, null);
     }

src/main/java/org/apache/datasketches/memory/internal/WritableBufferImpl.java

@@ -90,7 +90,7 @@ public static WritableBuffer wrapByteBuffer(
       type |= NONNATIVE_BO;
       wbuf = new NonNativeWritableBufferImpl(seg, type, memReqSvr, null);
     } else {
-      wbuf = new NativeWritableBufferImpl(null, seg, type, memReqSvr);
+      wbuf = new NativeWritableBufferImpl(seg, type, memReqSvr, null);
     }
     wbuf.setStartPositionEnd(0, byteBuffer.position(), byteBuffer.limit());
     return wbuf;