RFR: 8275687: runtime/CommandLine/PrintTouchedMethods test shouldn't catch RuntimeException

Ioi Lam iklam at openjdk.java.net
Thu Nov 25 07:00:05 UTC 2021


On Wed, 24 Nov 2021 12:48:37 GMT, Fairoz Matte <fmatte at openjdk.org> wrote:

> During the review of https://github.com/openjdk/jdk/pull/5231#pullrequestreview-784995993 
> It was suggested by @iklam in the test case Runtime exception is not required for the reason TouchedMethodsDCmd is always enabled as long as the current JVM has INCLUDE_SERVICES is true.
> 
> Igor has agreed to capture this in separate RFE (current) to handle this scenario.
> 
> This is trivial, please review.

LGTM and I agree this is trivial

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

Marked as reviewed by iklam (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6536


More information about the hotspot-runtime-dev mailing list