Integrated: CODETOOLS-7903262: JOL: GraphStatsWalker counts array elements incorrectly

Vladimir Plizga duke at openjdk.org
Mon Aug 29 05:51:32 UTC 2022


On Thu, 25 Aug 2022 04:32:58 GMT, Vladimir Plizga <duke at openjdk.org> wrote:

> #### Observed behavior
> Current `GraphStatsWalker` implementation does not count size and quantity of objects directly contained in arrays.
> 
> #### Desired behavior
> The `totalSize()` and `totalCount()` methods of a `GraphStats` instance should return the same values as the ones of `GraphLayout` instance built for the same object.
> 
> #### Additional info
> There is a [simple unit test](https://gist.github.com/Toparvion/adec9052ad23e85e54d72b5bfb5a1aa3) that can be used to reproduce current behavior without checking out the fix branch.

This pull request has now been integrated.

Changeset: 76c554a0
Author:    Vladimir Plizga <toparvion at gmx.com>
Committer: Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.org/jol/commit/76c554a0f8fac92e00e4a887808201eb48c5c905
Stats:     19 lines in 2 files changed: 19 ins; 0 del; 0 mod

7903262: JOL: GraphStatsWalker counts array elements incorrectly

Reviewed-by: shade

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

PR: https://git.openjdk.org/jol/pull/33


More information about the jol-dev mailing list