Minor improvement for readBuffer interop message

timfel · timfel · commit 93882fc66def · 2025-06-26T13:07:28.000+02:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/PythonAbstractObject.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/PythonAbstractObject.java
@@ -2802,9 +2802,7 @@ public void readBuffer(long byteOffset, byte[] destination, int destinationOffse
                 throw InvalidBufferOffsetException.create(byteOffset, length);
             }
             int offset = toIntNode.executeWithThrow(inliningTarget, byteOffset, raiseNode, PythonBuiltinClassType.OverflowError);
-            for (int i = 0; i < length; i++) {
-                destination[destinationOffset + i] = bufferLib.readByte(this, offset + i);
-            }
+            bufferLib.readIntoByteArray(this, offset, destination, destinationOffset, length);
         } else {
             throw UnsupportedMessageException.create();
         }

Original file line number	Diff line number	Diff line change
`@@ -2802,9 +2802,7 @@ public void readBuffer(long byteOffset, byte[] destination, int destinationOffse`
`2802`	`2802`	`throw InvalidBufferOffsetException.create(byteOffset, length);`
`2803`	`2803`	`}`
`2804`	`2804`	`int offset = toIntNode.executeWithThrow(inliningTarget, byteOffset, raiseNode, PythonBuiltinClassType.OverflowError);`
`2805`		`- for (int i = 0; i < length; i++) {`
`2806`		`- destination[destinationOffset + i] = bufferLib.readByte(this, offset + i);`
`2807`		`- }`
	`2805`	`+ bufferLib.readIntoByteArray(this, offset, destination, destinationOffset, length);`
`2808`	`2806`	`} else {`
`2809`	`2807`	`throw UnsupportedMessageException.create();`
`2810`	`2808`	`}`