RFR: 8134626 - Misc cleanups after generation array removal
david.lindholm at oracle.com
Mon Aug 31 13:22:04 UTC 2015
I think these changes looks good. There are changes to quite a lot of
files, especially in increment 2. I don't know if backporting fixes
later on will be a problem? Also, some of these files are in runtime,
for example vmStructs.cpp. Maybe you should let someone from runtime
review as well?
That said, I think these are nice cleanups.
On 2015-08-28 16:37, Jesper Wilhelmsson wrote:
> Please review this rather large cleanup. While working with the
> generation array removal I did a bunch of unrelated cleanups in the
> code I passed by. I split this out into a separate patch to keep the
> other changes clean.
> This change is split into three webrevs to make it easier to review. I
> intend to push all three as one change.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8134626
> Increment 1: Renaming
> Changes the variable name gen to young_gen or old_gen depending on
> what the variable will contain. The few cases where the name gen is
> untouched are places where the variable actually can contain either
> Increment 2: Comments and indentation
> * Fixes up alignment in lots of places.
> * Inserts and removes empty lines where needed.
> * Cleans up comments where we previously talked about older and
> younger generations. Fixes a few typos and clarifies some comments
> with respect to young/old generations and collections.
> * Added spaces around some operators.
> Increment 3: Code changes
> * Merged strings that was split on several lines for no good reason.
> * Added braces for if statements and for loops.
> * Removed dead code.
> * Moved variable initialization to initializer list in CollectedHeap
> * Updated flag descriptions "youngest" -> "young".
More information about the hotspot-gc-dev