RFR: 8249225 - Move definition of PADDING_ELEM_NUM
Kim Barrett
kim.barrett at oracle.com
Thu Jul 23 10:51:08 UTC 2020
> On Jul 23, 2020, at 6:24 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>
> 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.
Looks good to me too, with that re-added newline.
> Do you need a sponsor?
>
> Thanks,
> Thomas
More information about the hotspot-gc-dev
mailing list