RFR: 8283803: Remove jtreg tag manual=yesno for java/awt/print/PrinterJob/PrintGlyphVectorTest.java and fix test [v9]

lawrence.andrews duke at openjdk.java.net
Tue Apr 19 19:48:28 UTC 2022


On Tue, 19 Apr 2022 18:47:22 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> lawrence.andrews has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Merge branch 'JDK-8283803' of https://github.com/lawrence-andrew/jdk into JDK-8283803
>>  - Removed checking pj for null
>
> test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 123:
> 
>> 121:         instructionsText.setLineWrap(true);
>> 122: 
>> 123:         long tTimeout = TimeUnit.MINUTES.toMillis(testTimeOut);
> 
> Here you could probably reused `testTimeout`:
> 
> testTimeOut = TimeUnit.MINUTES.toMillis(testTimeOut);

testTimeOut is used with in lambda hence it should be final. So I just created a new variable.

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

PR: https://git.openjdk.java.net/jdk/pull/8004



More information about the client-libs-dev mailing list