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