On Fri, 16 Jul 2021 04:04:13 GMT, Prasanta Sadhukhan <psadhuk...@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

Reply via email to