RFR: 8227168: Cleanup usage of NEW_C_HEAP_ARRAY
kim.barrett at oracle.com
Mon Sep 2 18:52:54 UTC 2019
> On Sep 2, 2019, at 8:23 AM, Leo Korinth <leo.korinth at oracle.com> wrote:
> After I got caught doing an unnecessary check on the return value of NEW_C_HEAP_ARRAY (a mistake that I copied) I thought it would be good to do a cleanup in the sources so that others would not fall into this trap. This is the result.
> I have removed some places where the VM will be shut down after NEW_C_HEAP_ARRAY returns NULL (it never does return NULL, it does instead exit). I have also removed lots of unnecessary casts, that might hide bugs.
> There are /numerous/ buggy usage that I have NOT fixed. Those are bugs that ought to be fixed, but they are not in the gc folder, and I am unsure about the best solution (the do not just directly exit the VM). I have created a BUG to track them (https://bugs.openjdk.java.net/browse/JDK-8230395), and I will leave them for others to solve.
> I have changed some code paths that I can not test (AIX for example) so it would be nice if someone could verify it works there.
> mach5 tier1-3 (solaris, linux, windows, mac)
Looks good. Yay for unnecessary cast removals!
More information about the hotspot-dev