cleanup

Author: riccardobl

jme-angle/src/native/angle

@@ -992,7 +992,7 @@ public static void readToByteArray(InputStream input, byte[] dst, int bytesToRea
      * @param byteOffset     start offset within source (relative to beginning)
      * @param byteStride     stride in bytes (0 means tightly packed = element size)
      * @param numComponents  components per element (e.g. 3 for VEC3)
-     * @param originalFormat the source component type (from glTF accessor.componentType)
+     * @param originalFormat the source component type  
      * @param targetFormat   the desired buffer view type to return
      */
     public static Buffer getBufferView(ByteBuffer source, int byteOffset,  int count, int byteStride,
@@ -1008,15 +1008,11 @@ public static Buffer getBufferView(ByteBuffer source, int byteOffset,  int count
         int start = byteOffset;
         int bytes = stride * count;
 
-        // Full-range duplicate so absolute offsets are correct
-        ByteBuffer dup = source.duplicate();
-        dup.clear(); // pos=0, limit=capacity
 
         boolean tightlyPacked = (stride == elemSize);
 
         if (tightlyPacked) {
-            // Narrow the view to the accessor range and set LE order
-            ByteBuffer view = dup.duplicate();
+            ByteBuffer view = source.duplicate();
             view.position(start).limit(start + bytes);
             view = view.slice().order(ByteOrder.LITTLE_ENDIAN);
 
@@ -1027,7 +1023,7 @@ public static Buffer getBufferView(ByteBuffer source, int byteOffset,  int count
                     if (srcCompSize == 1 &&
                         (originalFormat == VertexBuffer.Format.Byte ||
                          originalFormat == VertexBuffer.Format.UnsignedByte)) {
-                        return view; // ByteBuffer shares memory
+                        return view;
                     }
                     break;
 
@@ -1060,7 +1056,6 @@ public static Buffer getBufferView(ByteBuffer source, int byteOffset,  int count
                     break;
 
                 case Double:
-                    // glTF doesn't use doubles; if ever needed, ensure 8-byte alignment
                     if (srcCompSize == 8 &&
                         originalFormat == VertexBuffer.Format.Double &&
                         (start & 7) == 0) {
@@ -1076,25 +1071,24 @@ public static Buffer getBufferView(ByteBuffer source, int byteOffset,  int count
             case Byte:
             case UnsignedByte: {
                 ByteBuffer out = BufferUtils.createByteBuffer(elements);
-                populateBuffer(out, dup, count, byteOffset, byteStride, numComponents, originalFormat);
+                populateBuffer(out, source, count, byteOffset, byteStride, numComponents, originalFormat);
                 return out;
             }
             case Short:
             case UnsignedShort: {
                 ShortBuffer out = BufferUtils.createShortBuffer(elements);
-                populateBuffer(out, dup, count, byteOffset, byteStride, numComponents, originalFormat);
+                populateBuffer(out, source, count, byteOffset, byteStride, numComponents, originalFormat);
                 return out;
             }
             case Int:
             case UnsignedInt: {
                 IntBuffer out = BufferUtils.createIntBuffer(elements);
-                populateBuffer(out, dup, count, byteOffset, byteStride, numComponents, originalFormat);
+                populateBuffer(out, source, count, byteOffset, byteStride, numComponents, originalFormat);
                 return out;
             }
             case Float: {
-                // Handles normalized integer sources via readAsFloat(...)
                 FloatBuffer out = BufferUtils.createFloatBuffer(elements);
-                populateBuffer(out, dup, count, byteOffset, byteStride, numComponents, originalFormat);
+                populateBuffer(out, source, count, byteOffset, byteStride, numComponents, originalFormat);
                 return out;
             }
             case Double: