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've updated the copyrights, using a script to do so.   

I eyeballed the latest diffs and they look OK.  That being said, one file stood 
out as having an incorrectly indented legal header block. It was 
TestReporterStreams.java.  I'll fix that separately, to avoid overloading this 
PR even more.

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

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

Reply via email to