RFR: 8367867: [macosx] Ignorable whitespace in text not removed when printing [v2]
Daniel Gredler
dgredler at openjdk.org
Wed Sep 17 17:29:14 UTC 2025
On Wed, 17 Sep 2025 17:22:01 GMT, Phil Race <prr at openjdk.org> wrote:
>> Done, thanks!
>
> You have removed the only override, and the method does nothing with anything except the provided string.
> So it seems to me this could be a static utility method, like isIgnorableWhitespace.
> I'm not suggesting moving it, just making it static.
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)?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27345#discussion_r2356229985
More information about the client-libs-dev
mailing list