On Thu, 15 Jan 2026 17:29:59 GMT, Phil Race <[email protected]> wrote:
>> Previous to this fix, no resolution was being found via CUPS/IPP and when
>> the test case for printer resolution was falling back to the made up 300dpi
>> setting. With this fix it finds the 600dpi from CUPS/IPP.
>
> Phil Race has updated the pull request incrementally with one additional
> commit since the last revision:
>
> 8375221
src/java.desktop/unix/classes/sun/print/IPPPrintService.java line 1711:
> 1709: for (int i = 0; i < numRes; i++) {
> 1710: int units = (rawResolutions[i*3+2] == 4) ?
> PrinterResolution.DPCM : PrinterResolution.DPI;
> 1711: pres[i] = new PrinterResolution(rawResolutions[i*3],
Suggestion:
pres[i] = new PrinterResolution(rawResolutions[i*3],
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29208#discussion_r2705154554
