RFR: 8359167: Remove unused test/hotspot/jtreg/vmTestbase/nsk/share/jpda/BindServer.java [v2]
Serguei Spitsyn
sspitsyn at openjdk.org
Tue Jun 17 22:36:29 UTC 2025
On Tue, 17 Jun 2025 17:49:11 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
>> I guess, Alex suggested to simplify this statement, or maybe to remove it.
>> This part looks a little bit strange when there is only one (local) mode:
>> `these methods corresponding to the mode`.
>
> There are a lot of references to "local" in comments and API names (in most of the files changed, not just DebugeeProcessl.java). I started down the path of cleaning these all up, but there ended up being a lot, and it distracts from the main purpose of this CR, which is removing BindServer.java. I'd rather do the other "local" cleanups in another PR.
It sounds good to me, thanks!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25785#discussion_r2153034919
More information about the serviceability-dev
mailing list