On Wed, 16 Nov 2022 03:44:24 GMT, Phil Race <[email protected]> wrote:
> This test is not problem listed and has failed only once since (SFAIK) it was
> last updated a year ago so
> this is just another small attempt to remove instability by increasing
> timeout, being really insistent about
> making sure the "done=true" is propagated and adding some extra timing
> logging to help analyse any failures
test/jdk/java/awt/FontClass/DrawStringWithInfiniteXform.java line 90:
> 88: timer.cancel();
> 89: }
> 90: System.out.println("Test passed");
It seems timer.cancel() says "Terminates this timer, discarding any currently
scheduled tasks." so it might disregard the "done" check been done in
ScheduleTask resulting in false positive...
Should we do Thread.yield() instead?
Also, this test has many coding violations like l70, l79 where no space between
operators,variables...
-------------
PR: https://git.openjdk.org/jdk/pull/11179
