On Fri, 10 Apr 2026 05:05:13 GMT, Sergey Bylokhov <[email protected]> wrote:

>> Set the cause of a `PrinterException` to the `RuntimeException` caught 
>> during printing, thus wrap any other exception that occurs during printer 
>> into a `PrinterException`.
>> 
>> I updated the test to print to a file—there's no need to send a print job to 
>> a physical printer. (Otherwise, the test asked me to select a file for the 
>> PDF Printer that I had.)
>
> src/java.desktop/share/classes/sun/print/RasterPrinterJob.java line 1421:
> 
>> 1419:             job.print(doc, attributes);
>> 1420:         } catch (PrintException e) {
>> 1421:             throw new PrinterException(e.toString());
> 
> Should we update this block as well?

It makes sense but under another bug id because this one is a regression of 
[JDK-8268675](https://bugs.openjdk.org/browse/JDK-8268675).

I'd like not to mix the changes.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/30656#discussion_r3063460977

Reply via email to