On Mon, 2 Nov 2026 19:17:13 GMT, Alexey Ushakov <[email protected]> wrote:
>> Hi @avu , I added the resultCode as suggested. Please check.
>> (I also added a little comment to maybe use/check the resultCode earlier
>> than now - where exactly in the for loop in ProcessPath() should I do it? Or
>> keep it as it is ?
>
> 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
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33513#discussion_r4446689110
