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