RFR: 8241574: Shenandoah: remove ShenandoahAssertToSpaceClosure
Charlie.Gracie at microsoft.com
Mon Aug 10 14:44:31 UTC 2020
My bad on the link. I used the title from the Bug for the webrev title. That ended up creating
an invalid filename due to the ":", I believe. I will make sure to verify the "Patch of changes:"
link in the future.
On 2020-08-10, 2:35 AM, "Aleksey Shipilev" <shade at redhat.com> wrote:
On 8/6/20 4:06 PM, Charlie Gracie wrote:
> Please review this small change that removes the unused ShenandoahAssertToSpaceClosure
> closure. JDK 8223774  removed the last use of this closure.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8241574
> Webrev: http://cr.openjdk.java.net/~burban/cgracie/remove_closure/webrev0.0/
- The upstream changes need to be reviewed at hotspot-gc-dev@, possibly cc'ing shenandoah-dev at .
- The webrev is misformatted. Notably, "Patch of changes:" is a garbage link, and _that_ is usually
what is used by sponsors to apply the whole change. I had to reassemble the patch from individual
More information about the hotspot-gc-dev