RFR: 8359958: Cleanup "local" debuggee references after JDK-8333117 removed support for non-local debuggees [v2]
Leonid Mesnik
lmesnik at openjdk.org
Mon Jun 30 22:35:39 UTC 2025
On Mon, 23 Jun 2025 17:15:29 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
>> test/hotspot/jtreg/vmTestbase/nsk/share/jpda/DebugeeProcess.java line 50:
>>
>>> 48: * @see nsk.share.jdwp.Debugee
>>> 49: */
>>> 50: public class DebugeeProcess {
>>
>> Are there any reason to make it non-abstract.. Even DebugeeProcess doesn't have abstract methods, only jid/jdwp subclasses might be used.Might be left it abstract?
>
> Currently it is only used via subclasses, but that doesn't necessarily mean it couldn't be used directly. Is there a reason you want to disallow that?
No any particular reason.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25887#discussion_r2176082458
More information about the serviceability-dev
mailing list