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

Fairoz Matte fmatte at openjdk.java.net
Thu Nov 25 08:41:08 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.

This pull request has now been integrated.

Changeset: f0136ec9
Author:    Fairoz Matte <fmatte at openjdk.org>
Committer: Christian Hagedorn <chagedorn at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/f0136ec94539d0e30ec11d44f8143196da1f7125
Stats:     6 lines in 1 file changed: 0 ins; 4 del; 2 mod

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

Reviewed-by: iklam, chagedorn

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

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


More information about the hotspot-runtime-dev mailing list