RFR: 8309510: com/sun/jdi/RedefineNestmateAttr/TestNestmateAttr.java no longer needs to override startup() method
Chris Plummer
cjplummer at openjdk.org
Tue Jun 6 00:44:08 UTC 2023
com/sun/jdi/RedefineNestmateAttr/TestNestmateAttr.java currently overrides the TestScaffold.startup() method:
// override this to correct a bug so arguments can be passed to
// the Target class
protected void startUp(String targetName) {
List<String> argList = new ArrayList<>(Arrays.asList(args));
argList.add(0, targetName); // pre-pend so it becomes the first "app" arg
println("run args: " + argList);
connect((String[]) argList.toArray(args));
waitForVMStart();
}
This issue of passing app args was fixed recently by [JDK-8308481](https://bugs.openjdk.org/browse/JDK-8308481), so the override is no longer needed.
-------------
Commit messages:
- No longer need to override starupt() after JDK-8308481 fix.
Changes: https://git.openjdk.org/jdk/pull/14325/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=14325&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8309510
Stats: 10 lines in 1 file changed: 0 ins; 10 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/14325.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/14325/head:pull/14325
PR: https://git.openjdk.org/jdk/pull/14325
More information about the serviceability-dev
mailing list