On Wed, 17 Sep 2025 14:47:49 GMT, Daniel Gredler <[email protected]> wrote:
>> src/java.desktop/share/classes/sun/print/RasterPrinterJob.java line 2479:
>>
>>> 2477: * don't have to worry about platform or font specific handling
>>> of them.
>>> 2478: */
>>> 2479: public String removeControlChars(String s) {
>>
>> Perhaps the commend on this method could be updated too, since the method
>> became `public`?
>
> 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.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27345#discussion_r2356217545
