On Thu, 17 Jul 2025 08:00:15 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8362452
>
> src/java.desktop/macosx/classes/sun/lwawt/macosx/CPrinterJob.java line 661:
> 
>> 659: 
>> 660:     private static native long createNSPrintInfo();
>> 661:     private static native void disposeNSPrintInfo(long printInfo);
> 
> What is the reason for using static here when it was not before?

because it has no need for the instance and moreover it can't be called on the 
instance because that would mean it is still reachable.
It is called instead from the disposer.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26359#discussion_r2214310791

Reply via email to