On Fri, 16 Jul 2021 21:26:52 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

> Yes, this null check is needed, since we specify that setting the `printer` 
> property to `null` will use the default printer.


oh yeah. So nothing to do here.

The only question is about the behaviour of
 public static final PrinterJob createPrinterJob(Printer printer)  {}
when called with null.

- Document the NPE
- Don't do anything 
-  Substitute the default printer and create the job and document this
- return null if there isn't one and document this.
-

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

PR: https://git.openjdk.java.net/jfx/pull/574

Reply via email to