On Fri, 13 Jun 2025 23:37:32 GMT, Alex Menkov <amen...@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.
>
> test/hotspot/jtreg/vmTestbase/nsk/share/jpda/DebugeeProcess.java line 44:
> 
>> 42:  * debugee VM process.
>> 43:  * Derived classes should implement these methods corresponding to the 
>> mode
>> 44:  * that the process should be started in (locally).
> 
> I think this can be dropped

Which part?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25785#discussion_r2150376249

Reply via email to