[jdk8u-dev] RFR: 8071507: (ref) Clear phantom reference as soft and weak references do [v2]

Kim Barrett kbarrett at openjdk.org
Tue Aug 9 03:51:27 UTC 2022


On Fri, 5 Aug 2022 22:28:47 GMT, Mandy Chung <mchung at openjdk.org> wrote:

> Is there a reason for backporting two changes in one PR? I see this was done in 8u42 as well. It makes it harder to review than necessary, notably where the two change the same code in `referenceProcessor.cpp`.

There was a change before 8071507 in mainline (and backported to jdk8u) that wasn’t in the RI. That change was rendered moot (the changed code was entirely removed) by 8143847. So if we’d separated 8071507 and 8143847 in the RI we would have needed to first apply that other change to the RI (8071507 without that earlier change would have been wrong), only to have it disappear with 8143847.

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

PR: https://git.openjdk.org/jdk8u-dev/pull/94


More information about the jdk8u-dev mailing list