On Thu, 5 Mar 2026 21:43:35 GMT, Daniel Gredler <[email protected]> wrote:

> FontInfo contains a few small items which could use fixing:
> 
> - toString() throws NPE if `devTx` or `glyphTx` are null (I ran into this a 
> couple of times during debugging when my IDE tried to display the string 
> representation)
> 
> - toString() is missing a comma and space between the last two attributes
> 
> - Two overridden methods do not have the `@Override` annotation

LGTM

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

Marked as reviewed by prr (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/30098#pullrequestreview-3899948384

Reply via email to