Fix: Add missing return statement to CDataObject#writeByte.

phoeinx · phoeinx · commit 0d6e40dc47cd · 2022-01-10T11:58:41.000+01:00
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/ctypes/CDataObject.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/ctypes/CDataObject.java
@@ -136,6 +136,7 @@ void writeByte(int byteIndex, byte value,
                     @CachedLibrary(limit = "2") PythonBufferAccessLibrary bufferLib) {
         if (b_ptr.ptr != null) {
             bufferLib.writeByte(b_ptr.ptr, b_ptr.offset + byteIndex, value);
+            return;
         }
         throw CompilerDirectives.shouldNotReachHere("buffer write to empty CDataObject");
     }

Original file line number	Diff line number	Diff line change
`@@ -136,6 +136,7 @@ void writeByte(int byteIndex, byte value,`
`136`	`136`	`@CachedLibrary(limit = "2") PythonBufferAccessLibrary bufferLib) {`
`137`	`137`	`if (b_ptr.ptr != null) {`
`138`	`138`	`bufferLib.writeByte(b_ptr.ptr, b_ptr.offset + byteIndex, value);`
	`139`	`+ return;`
`139`	`140`	`}`
`140`	`141`	`throw CompilerDirectives.shouldNotReachHere("buffer write to empty CDataObject");`
`141`	`142`	`}`