On Wed, 21 Jan 2026 18:52:00 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

LGTM

src/java.desktop/unix/classes/sun/print/AttributeClass.java line 198:

> 196:             res[2] = (int)bufArray[9];
> 197:         }
> 198:         return res;

other files copyright year updated except this so pointing it out..

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

Marked as reviewed by psadhukhan (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29208#pullrequestreview-3695513237
PR Review Comment: https://git.openjdk.org/jdk/pull/29208#discussion_r2719440086

Reply via email to