On Sun, 22 Nov 2020 22:15:20 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:
> I didn't notice this before it was integrated. > > The test for inactive isn't right; rather than `next == this` it > should be `next != null`. This becomes apparent once > FinalizerHistogram is fixed to call getInactive() rather than get(). > > I noticed this while working on JDK-8256517, where I ran into some > similar issues. I will address these problems as part of that change. Oh ok. Thanks for taking care of it! ------------- PR: https://git.openjdk.java.net/jdk/pull/1231