On Wed, 17 Sep 2025 19:28:10 GMT, Phil Race <[email protected]> wrote:

>> I actually wondered about maybe moving this (as static) to `FontUtilities`, 
>> next to (and as a sort of companion to) `isIgnorableWhitespace`). Do you 
>> think that would be a good change? Or better to just leave it in 
>> `RasterPrinterJob` (as static)?
>
> My comment about "not moving it" was meant as a way of saying I'd thought 
> about FontUtilities but since it is only used by printing keep it here 
> unless/until we find a reason to use it in a wider context.

Done, thanks!

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27345#discussion_r2359748833

Reply via email to