Integrated: CODETOOLS-7903302: cleanup code

Jonathan Gibbons jjg at openjdk.org
Wed Sep 14 04:25:12 UTC 2022


On Tue, 13 Sep 2022 22:05:28 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:

> Please review a medium-sized update to clean up the jtreg code.
> 
> Most of the changes were performed directly by the IDE. Many of the changes are very local (same line) but some affect surrounding lines when the code was reformatted ... for example, when using lambdas, or try with resources.
> 
> The most "risky" change was converting `Arrays.asList`, which permits `null`, to `List.of`, which does not. This was fixed up manually in a few places, most notable in `ModuleConfig`. In all places, `null` was easily changed to an empty string `""`.

This pull request has now been integrated.

Changeset: f081a45f
Author:    Jonathan Gibbons <jjg at openjdk.org>
URL:       https://git.openjdk.org/jtreg/commit/f081a45fbebdefcada3f58fff86d9d6e835a7921
Stats:     517 lines in 30 files changed: 12 ins; 258 del; 247 mod

7903302: cleanup code

Reviewed-by: iris

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

PR: https://git.openjdk.org/jtreg/pull/118


More information about the jtreg-dev mailing list