On Thu, 25 Sep 2025 20:45:35 GMT, Phil Race <[email protected]> wrote:

> The only thing I ask is that we annotate PrintTextTest.java with this bug id 
> in the @bug tag

@prrace  Done.

> It seems PrintTextTest catches too many issues, can we create a small test to 
> cover just this patch?

@mrserb I tried writing a smaller automated test for this change, using 
PostScript to make it possible to "print" without printing and then roughly 
extract the bounds of the text from the generated PostScript, but it doesn't 
exercise the code in `CTextPipe.m`. I think this requires the manual testing 
provided by `PrintTextTest`.

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

PR Comment: https://git.openjdk.org/jdk/pull/27486#issuecomment-3336202596

Reply via email to