<AWT Dev> RFR: 8257853: Remove dependencies on JNF's JNI utility functions in AWT and 2D code [v6]

[Phil Race]

On Fri, 18 Dec 2020 02:30:47 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8257853: Remove dependencies on JNF's JNI utility functions in AWT and 2D code
>
> src/java.desktop/macosx/native/libosxapp/JNIUtilities.h line 120:
> 
>> 118:      } \
>> 119:      LOG_NULL(dst_var, name); \
>> 120:      CHECK_NULL(dst_var);
> 
> If some of these macros will be used on appkit thread, then the check like "CHECK_NULL" will not work,  because the code will never return to the java, and the failure will be actually ignored.

If we get NULL back then there's a progamming bug. LOG_NULL will show it.
CHECK_NULL is just as much to satisfy JNI as anything.
If we are on a Java thread then it will still add some value so I don't see the point in removing it.
Nor do I think I'd like to use NSException instead because that affects the flow of control.
Yes, this may be different than before (probably more relevant to the other comment) but I think better may be a better approach overall. If we ever see an issue here that is likely to occur due to anything other than a bad internal inconsistency due to JDK changes then I would prefer to follow the code path and fix it up.

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

PR: https://git.openjdk.java.net/jdk/pull/1679