On Mon, 2 Nov 2026 19:19:09 GMT, Alexey Ushakov <[email protected]> wrote:
>> I thought about adding it to the loop that processes segments of the path
>> `jboolean ProcessPath(ProcessHandler* hnd,`
>> ` jfloat transXf, jfloat transYf,`
>> ` jfloat* coords, jint maxCoords,`
>> ` jbyte* types, jint numTypes)`
>> {
>> ` for (i = 0; i < numTypes; i++) {`
>> ` switch (types[i]) {`
>> `...`
>> ` }`
>> ` /* Check and if fail return JNI_FALSE */`
>> ` }`
>> `...`
>
> So, you'll have the error handling logic from ProcessPath result code
I moved the resultCode check to `ProcessPath(...)` .
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33513#discussion_r4481248965