RFR (S): 8079091: Remove dictionary NULL check on common path of BlockFreeList methods

Kim Barrett kim.barrett at oracle.com
Thu Apr 30 19:20:20 UTC 2015


On Apr 30, 2015, at 3:15 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
> 
> If you *really* want to keep the accessor function, then there
> probably needs to be two versions of it, one const and returning
> const*, and the other non-const and (probably) returning non-const*.

A different solution would be to declare the subobject mutable and
have one const qualified accessor returning non-const.  But that's
also feeling odd compared to just using the member.  This is all code
within the class having the member.  The accessor isn't public, and
it's implementation is trivial; I just don't see the point.




More information about the hotspot-runtime-dev mailing list