On Fri, 18 Dec 2020 11:11:52 GMT, Prasanta Sadhukhan <psadhuk...@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

Reply via email to