RFR: 8356811: Some nsk/jdi tests can fetch ThreadReference from static field in the debuggee: part 4 [v2]

Chris Plummer cjplummer at openjdk.org
Fri May 16 20:10:52 UTC 2025


On Fri, 16 May 2025 02:52:47 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> @alexmenkov It looks like this could be done in almost every test in this PR. In some the `try` just needs to move about 8 lines up, but in others it's more like 20. There will be a lot of indentation diffs. Do you still want to see this change?
>
> It looks like there were some changes in part 3 that could also benefit from this. Perhaps this would be better off done in a followup PR. It will keep this PR easier to review, will make moving the `try` easier to review, and will pickup some additional tests that could use this.

I filed [JDK-8357172](https://bugs.openjdk.org/browse/JDK-8357172)

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25190#discussion_r2093619239


More information about the serviceability-dev mailing list