Integrated: 8271726: JFR: should use equal() to check event fields in tests

Denghui Dong ddong at openjdk.java.net
Wed Aug 4 01:38:31 UTC 2021


On Tue, 3 Aug 2021 11:14:29 GMT, Denghui Dong <ddong at openjdk.org> wrote:

> Hi,
> 
> Please help review this trivial test fix.
> 
> In TestHeapDump.java, we should use EventField.equal() to check fields, but there are two lines that didn't abide by this rule(I think it's a typo).
> 
> No other similar mistake was found in other jfr tests.
> 
> Thanks,
> Denghui

This pull request has now been integrated.

Changeset: 43a6ba96
Author:    Denghui Dong <ddong at openjdk.org>
Committer: Yi Yang <yyang at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/43a6ba9677551762c1bc1dc2f994dde7d6d9f1a5
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

8271726: JFR: should use equal() to check event fields in tests

Reviewed-by: egahlin

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

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


More information about the hotspot-jfr-dev mailing list