Integrated: 8309510: com/sun/jdi/RedefineNestmateAttr/TestNestmateAttr.java no longer needs to override startUp() method

Chris Plummer cjplummer at openjdk.org
Wed Jun 7 18:00:33 UTC 2023


On Tue, 6 Jun 2023 00:36:12 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: c38abbfc
Author:    Chris Plummer <cjplummer at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/c38abbfcaa80b3bb38bd6fc284e6bc3437199d77
Stats:     10 lines in 1 file changed: 0 ins; 10 del; 0 mod

8309510: com/sun/jdi/RedefineNestmateAttr/TestNestmateAttr.java no longer needs to override startUp() method

Reviewed-by: sspitsyn, amenkov

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

PR: https://git.openjdk.org/jdk/pull/14325


More information about the serviceability-dev mailing list