RFR: 8273921: Refactor NSK/JDI tests to create thread using factory [v2]

Alex Menkov amenkov at openjdk.java.net
Mon Sep 20 22:58:40 UTC 2021


On Mon, 20 Sep 2021 22:55:02 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

>> 8273921: Refactor NSK/JDI tests to create thread using factory
>
> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Removed some unused tName variables.

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassExclusionFilter/filter001a.java line 161:

> 159:     static class Thread1filter001a extends JDITask {
> 160: 
> 161:         String tName = null;

One more case. tName is not removed.

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassExclusionFilter/filter002a.java line 179:

> 177:     static class Thread2filter002a extends JDITask {
> 178: 
> 179:         String tName = null;

tName is not removed.

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassFilter_rt/filter_rt001a.java line 205:

> 203: class Thread2filter_rt001a extends JDITask {
> 204: 
> 205:     String tName = null;

tName is not removed.

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassFilter_rt/filter_rt002a.java line 205:

> 203: class Thread2filter_rt002a extends JDITask {
> 204: 
> 205:     String tName = null;

tName is not removed.

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassFilter_rt/filter_rt005a.java line 234:

> 232: class Thread2filter_rt005a extends JDITask {
> 233: 
> 234:     String tName = null;

The same

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassFilter_rt/filter_rt006a.java line 234:

> 232: class Thread2filter_rt006a extends JDITask {
> 233: 
> 234:     String tName = null;

One more case

test/hotspot/jtreg/vmTestbase/nsk/jdi/WatchpointRequest/addClassFilter_s/filter_s001a.java line 223:

> 221: class Thread2filter_s001a extends JDITask {
> 222: 
> 223:     String tName = null;

tName is not removed.

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

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


More information about the serviceability-dev mailing list