On Wed, 17 Sep 2025 13:28:57 GMT, Daniel Gredler <[email protected]> wrote:
> When text is printed (or drawn to screen), some whitespace characters like
> new lines should be ignored (see `FontUtilities.isIgnorableWhitespace(int)`).
> This is currently not being done when printing on macOS, and is one (of about
> half a dozen) reasons why `java/awt/print/PrinterJob/PrintTextTest.java` is
> currently problem listed on macOS. Fixing this bug will get us one step
> closer to removing this test from the problem list.
>
> Within `PrintTextTest`, this issue can be seen specifically with the extra
> space before and after "drawChars(" on every page of the printed document.
> This whitespace should not be present.
>
> This change aligns behavior with printing on other platforms (see
> `RasterPrinterJob.removeControlChars(String)`) and with screen display on
> macOS (see `CCharToGlyphMapper`) and other platforms.
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`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27345#discussion_r2355673235
