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:28 UTC 2025


On Tue, 17 Jun 2025 17:53:45 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> The nsk BindServer class is no longer used so it can be removed. This appears to be a remnant left over after the following recent change:
>> 
>> [JDK-8333117](https://bugs.openjdk.org/browse/JDK-8333117) Remove support of remote and manual debuggee launchers
>> 
>> There is also some additional cleanup of "manual" and "remote" debuggee launchers done by this PR and fixing some comments that have been wrong for a long time regarding "see also".
>> 
>> Tested by running nsk/jdi, nsk/jdb, and nsk/jdwp tests locally. I'll also run them with tier5 CI once reviews have been done.
>
> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update test/hotspot/jtreg/vmTestbase/nsk/share/jpda/IOPipe.java
>   
>   Co-authored-by: Alex Menkov <alexey.menkov at oracle.com>

This looks good.

I was not able to post any comment or approve for a couple of hours. It was some kind of `github` failure.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/25785#pullrequestreview-2936868210
PR Comment: https://git.openjdk.org/jdk/pull/25785#issuecomment-2982020892


More information about the serviceability-dev mailing list