On Tue, 3 Sep 2024 08:06:14 GMT, Kevin Walls <kev...@openjdk.org> wrote:
> Deprecation annotations and warnings on starting the tool(s).
> Handle man page in a separate issue.
src/jdk.hotspot.agent/doc/index.html line 28:
> 26: <p>
> 27: <strong>WARNING: debugd is deprecated and will be removed in a future
> release.</strong>
> 28: <p>
Suggestion:
<p><strong>WARNING: <b>jhsdb debugd</b> is deprecated and will be removed in a
future release.</strong></p>
<p>
src/jdk.hotspot.agent/doc/transported_core.html line 24:
> 22: the core dump, and then attaching to it from another machine by using
> the jhsdb
> 23: <b>--connect</b> argument. See the <b>jhsdb</b> man page for details.
> 24: <p><strong>WARNING: debugd is deprecated and will be removed in a
> future release.</strong></p>
Suggestion:
<p><strong>WARNING: <b>jhsdb debugd</b> is deprecated and will be removed
in a future release.</strong></p>
src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/DebugServer.java line 31:
> 29: import sun.jvm.hotspot.oops.*;
> 30:
> 31: @Deprecated(since="24", forRemoval=true)
I didn't even realize this class existed. So this an (undocumented) way to
start the debug server. Is there some other class that connects to it other
than using `jhsdb --connect`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20830#discussion_r1742608450
PR Review Comment: https://git.openjdk.org/jdk/pull/20830#discussion_r1742607327
PR Review Comment: https://git.openjdk.org/jdk/pull/20830#discussion_r1742623776
