RFR: JDK-8333360 : PrintNullString.java doesn't use float arguments

Abhishek Kumar abhiscxk at openjdk.org
Wed Jun 5 16:05:58 UTC 2024


On Wed, 5 Jun 2024 12:20:00 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> test/jdk/java/awt/print/PrinterJob/PrintNullString.java line 172:
>> 
>>> 170: 
>>> 171:             try {
>>> 172:                 g2d.drawString(emptyIterator, 20.0f, 180.0f);
>> 
>> Does the line below also need to change ?
>> `g2d.drawString("FAILURE: No IAE for empty iterator, float", 20.0f 180.0f);`
>
> No, it does not. It may though.
> 
> The test declares it verifies `drawString(Iterator, float, float)` — the test does it now. How the error message is output on the screen isn't important: either `int` or `float` version will do the same thing.

@aivanov-jdk Thank you for clarification.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19540#discussion_r1628050634


More information about the client-libs-dev mailing list