On Wed, 21 Dec 2022 16:47:42 GMT, Jonathan Gibbons <[email protected]> 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

Reply via email to