RFR: 8252000: remove usage of PropertyResolvingWrapper in vmTestbase/nsk/jdb

Aleksey Shipilev shade at openjdk.java.net
Fri Sep 25 05:40:29 UTC 2020


On Thu, 24 Sep 2020 23:55:15 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> the patch
> - removes `PropertyResolvingWrapper` from `vmTestbase/nsk/jdb` tests
> - updates `JdbArgumentHandler` to remove `"` from `jdb.option` option
> - reformats code
> 
> testing:
> ✅ `vmTestbase/nsk/jdb` on {macosx,linux,windows}-x64

The changes look good, but I think the practice of doing a massive cleanup in "remove usage of PropertyResolvingWrapper
in vmTestbase/nsk/jdb" PR is misleading. Better to split this in two issues (at least next time)?

test/hotspot/jtreg/vmTestbase/nsk/jdb/clear/clear004/clear004.java line 123:

> 121:         Vector<String> v;
> 122:
> 123:         v = new Vector<>();

Coalesce these two lines into `Vector<String> v = new Vector<>()`?

test/hotspot/jtreg/vmTestbase/nsk/jdb/clear/clear004/clear004.java line 142:

> 140:         Vector<String> v;
> 141:
> 142:         v = new Vector<>();

`Vector<String> v = new Vector<>()`?

test/hotspot/jtreg/vmTestbase/nsk/jdb/locals/locals002/locals002.java line 109:

> 107:             {"doubleVar", "2.578", "3.8976"},
> 108:             {"objVar", "objVarString", "objArgString"},
> 109:             {"arrVar", "int[5]", "int[3]"}

This table looked a bit better before. Maybe commas should move to the left, but tabbing restored?

test/hotspot/jtreg/vmTestbase/nsk/jdb/regression/b4689395/b4689395.java line 147:

> 145:     public b4689395() {
> 146:         classFile = ClassLoadUtils.getRedefineClassFileName(DEBUGGEE_CLASS);
> 147:         if (classFile == null)

Braces, while we are changing these lines anyway?

test/hotspot/jtreg/vmTestbase/nsk/jdb/caught_exception/caught_exception002/caught_exception002.java line 114:

> 112:         Paragrep grep;
> 113:         int count;
> 114:         Vector<String> v = new Vector<>();

As long as we cleaning up the coding here, the declarations can be moved to the first use? Here... and everywhere.

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

Marked as reviewed by shade (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/350


More information about the serviceability-dev mailing list