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

Phil Race prr at openjdk.java.net
Fri Dec 18 21:37:00 UTC 2020


On Fri, 18 Dec 2020 11:11:52 GMT, Prasanta Sadhukhan <psadhukhan 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/libawt_lwawt/awt/CPrinterJob.m line 743:
> 
>> 741:     if (printerJob == NULL) return NO;
>> 742:     GET_NSPRINTINFO_METHOD_RETURN(NO)
>> 743:     NSPrintInfo* printInfo = (NSPrintInfo*)jlong_to_ptr((*env)->CallLongMethod(env, printerJob, sjm_getNSPrintInfo)); // AWT_THREADING Safe (known object)
> 
> CHECK_EXCEPTION?

I've fixed a bunch of these but some - like this one - do not need CHECK_EXCEPTION.
We detect a null return value and return directly to Java and we are better to propagate any theoretical
exception in such cases.

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

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


More information about the awt-dev mailing list