[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Integrated: 8293037: Remove DebuggerBase.writeBytes() and related code from SA
From:       Chris Plummer <cjplummer () openjdk ! org>
Date:       2022-08-30 16:27:21
Message-ID: Hm_lo5oE00IfGiKy5gyq-erbxx5owSZBn539Tx0bo0k=.84bf6714-3ba0-42c9-94f3-c45be121d732 () github ! com
[Download RAW message or body]

On Mon, 29 Aug 2022 18:54:58 GMT, Chris Plummer <cjplummer@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@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


[prev in list] [next in list] [prev in thread] [next in thread]