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