RFR: 8249225 - Move definition of PADDING_ELEM_NUM
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Jul 23 10:24:52 UTC 2020
Hi,
On 21.07.20 03:41, Cesar Soares Lucas wrote:
> Bug: https://bugs.openjdk.java.net/browse/JDK-8249225
>
> Webrev: http://cr.openjdk.java.net/~adityam/cesar/8249225/
>
> Please review the above linked patch for the mentioned bug. It's a small change
> that moves the definition of "PADDING_ELEM_NUM" to the only file that uses it.
> I also took the chance of converting it from a "#define" directive into a "const"
> declaration - I can revert it back to a #define if you think it's better.
>
>
could the change please re-add the newline after array_length? It is
easier to read if generation of the constants is slightly offset from
the other code.
Looks good otherwise. I do not need a re-review for this change.
Do you need a sponsor?
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list