RFR: JDK-8286311: remove boilerplate from use of runTests [v2]

Hannes Wallnöfer hannesw at openjdk.org
Thu Dec 22 11:06:50 UTC 2022


On Wed, 21 Dec 2022 16:47:42 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

>> Please review a test-only cleanup for JDK 21.
>> 
>> There are two mostly-automated (IDE) changes here:
>> 
>> 1. Remove redundant boiler-plate arguments from calls to `JavadocTester.runTests()`.  The use of a function to derive and provide a method-specific directory because unnecessary with JDK-
>> 2. Use `var` for the declaration of the `tester` variable found in the `main` method of most javadoc tests.
>> 
>> The changes were mostly done with IDEA _Replace in Files..._, with some manual fixup for some outlying cases found while checking the code.
>
> Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision:
> 
>   update copyrights

I wouldn't have insisted on updating the header dates. The fact that you did so in an automated way makes me feel better about the fact that I made you do so. Looks good!

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

Marked as reviewed by hannesw (Reviewer).

PR: https://git.openjdk.org/jdk/pull/11746


More information about the javadoc-dev mailing list