RFR: 8375221: Update code to get PrinterResolution from CUPS/IPP print service [v2]

Andrey Turbanov aturbanov at openjdk.org
Mon Jan 19 15:14:20 UTC 2026


On Thu, 15 Jan 2026 17:29:59 GMT, Phil Race <prr at openjdk.org> 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


More information about the client-libs-dev mailing list