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

Reply via email to