Integrated: 8293037: Remove DebuggerBase.writeBytes() and related code from SA

Chris Plummer cjplummer at openjdk.org
Tue Aug 30 16:27:21 UTC 2022


On Mon, 29 Aug 2022 18:54:58 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

> DebuggerBase.writeBytes() is not needed. It is only called by a number of other "write" methods, such as writeJBoolean(), but these methods are never called, so they can be removed along with writeBytes(). Lastly, writeBytes() calls writeBytesToProcess() which has no other callers, so it too can be removed. All it currently does is throw an exception. I imagine there may have been some use for this "write" capability 20+ years ago on Solaris/x86, or maybe it was part of future plans that never panned out. In any case, it's all just baggage now that can be removed.

This pull request has now been integrated.

Changeset: f285cea9
Author:    Chris Plummer <cjplummer at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/f285cea9a6cbf7e46840eab74094e9588af4e147
Stats:     118 lines in 7 files changed: 0 ins; 116 del; 2 mod

8293037: Remove DebuggerBase.writeBytes() and related code from SA

Reviewed-by: amenkov, kevinw

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

PR: https://git.openjdk.org/jdk/pull/10068


More information about the serviceability-dev mailing list