RFR: 8178499: Remove _ptr_ and _size_ infixes from align functions
thomas.schatzl at oracle.com
Tue Jul 4 11:56:20 UTC 2017
On Fri, 2017-06-30 at 11:40 +0200, Stefan Karlsson wrote:
> Hi all,
> Please review this patch to remove the _ptr_ and _size_ infixes from
> align functions.
> Rename functions (and corresponding macros) from:
> The patch builds upon the changes in:
looks good. I am not sure if I am happy about the renamings of the
non-global align* methods, as they at least sometimes seemed to have
been part of the item you wanted to align. E.g.
ReservedSpace::page_align_size_* to ReservedSpace::page_align_*.
There is at least one name clash because of that is resolved using the
"::" specifiers (in space.cpp:~670). Or maybe this can be resolved by
different naming in ReservedSpace.
Unless somebody else also objects, consider this as not a blocking
More information about the hotspot-dev