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