RFR: 8269638: Property methods, setters, and getters in printing API should be final [v4]
Kevin Rushforth
kcr at openjdk.java.net
Fri Jul 16 13:43:52 UTC 2021
On Fri, 16 Jul 2021 04:04:13 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
> looks good..
> However I have a different question...I was looking at printerProperty and I saw In l182, we are not checking for getDefaultPrinter() returns null or not but in l120, we do...Is it not required in l182?
It might be a good follow-on bug to skip lines 185-6 if `getDefaultPrinter()` is `null` (else it will NPE), but it's completely unrelated to this fix so wouldn't be done as part of this PR.
-------------
PR: https://git.openjdk.java.net/jfx/pull/574
More information about the openjfx-dev
mailing list