RFR: 8353955: nsk/jdi tests should be fixed to not always require includevirtualthreads=y [v4]

Chris Plummer cjplummer at openjdk.org
Wed May 14 17:12:52 UTC 2025


On Wed, 14 May 2025 08:19:02 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

> Looks okay in general. Many files still need a copyright update.

I'll update after the merge with 2 other PRs that this PR is dependent on.

> test/hotspot/jtreg/vmTestbase/nsk/jdi/EventRequestManager/accessWatchpointRequests/accwtchpreq002.java line 251:
> 
>> 249:         String lineForComm  = "lineForComm";
>> 250: 
>> 251:         //ThreadReference   mainThread = debuggee.threadByNameOrThrow("main");
> 
> Q: Do you want to keep this commented out line?

I thought I had undone that. This test is fixed in another PR, but I had experimented with the fix here first. I'll remove it.

> test/hotspot/jtreg/vmTestbase/nsk/share/jpda/DebugeeArgumentHandler.java.orig line 1:
> 
>> 1: /*
> 
> I guess, this temporary file was checked in by a mistake.

Good catch. At some point I did a "git add -a", not knowing that this filing was lingering around. I'll remove it.

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

PR Comment: https://git.openjdk.org/jdk/pull/24606#issuecomment-2880959336
PR Review Comment: https://git.openjdk.org/jdk/pull/24606#discussion_r2089384545
PR Review Comment: https://git.openjdk.org/jdk/pull/24606#discussion_r2089386062


More information about the serviceability-dev mailing list