Integrated: 8303151: DCmd framework cleanups

David Holmes dholmes at openjdk.org
Tue Mar 7 22:58:19 UTC 2023


The message from this sender included one or more files
which could not be scanned for virus detection; do not
open these files unless you are certain of the sender's intent.

----------------------------------------------------------------------
On Fri, 3 Mar 2023 04:59:44 GMT, David Holmes <dholmes at openjdk.org> wrote:

> Whilst working on the DCmd code I noticed two items that could be cleaned up:
> 
> 1. The `NMTDCmd` is registered after the call to `register_dcmds()` instead of inside it.
> 
> 2. The "extension" mechanism to define external DCmds (as added by [JDK-7132515](https://bugs.openjdk.org/browse/JDK-7132515) for `UnlockCommercialFeatures`) is no longer needed.
> 
> Testing: tiers 1-3
> 
> Thanks

This pull request has now been integrated.

Changeset: 5f1108f8
Author:    David Holmes <dholmes at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/5f1108f8f0768837591b06d47dec857963ed1fcb
Stats:     32 lines in 3 files changed: 6 ins; 23 del; 3 mod

8303151: DCmd framework cleanups

Reviewed-by: jsjolen, stuefe, yyang

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

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


More information about the serviceability-dev mailing list