RFR: 8362452: [macOS] Remove CPrinterJob.finalize() [v2]
Phil Race
prr at openjdk.org
Thu Jul 17 21:16:04 UTC 2025
On Thu, 17 Jul 2025 08:00:15 GMT, Prasanta Sadhukhan <psadhukhan at 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
More information about the client-libs-dev
mailing list