On Wed, 11 Jun 2025 19:00:45 GMT, Sergey Bylokhov <s...@openjdk.org> wrote:

>> src/java.base/share/native/libnet/net_util.c line 77:
>> 
>>> 75:     CHECK_NULL_RETURN(s, JNI_VERSION_1_2);
>>> 76:     preferIPv4Stack = (*env)->CallStaticBooleanMethod(env, iCls, mid, 
>>> s);
>>> 77:     JNU_CHECK_EXCEPTION_RETURN(env, JNI_VERSION_1_2);
>> 
>> The exception check is technically not necessary because no other JNI 
>> methods are called before the function exits, but it wasn't always the case.
>
> If this check is not needed, then why are we adding it?

just so we don't forget to add it when we modify the enclosing function. The 
call to `CallStaticBooleanMethod` is located pretty far from the end of the 
function, and can be easy to miss.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25754#discussion_r2142645794

Reply via email to