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