RFR: 8204585: Remove IN_ARCHIVE_ROOT from Access API

Kim Barrett kim.barrett at oracle.com
Tue Jun 12 14:46:51 UTC 2018


> On Jun 12, 2018, at 4:11 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> 
> Hi,
> 
> On Mon, 2018-06-11 at 18:58 -0400, Kim Barrett wrote:
>>> On Jun 11, 2018, at 6:26 PM, Jiangli Zhou <jiangli.zhou at oracle.com>
>>> wrote:
>>> 
>>> Hi Kim,
>>> 
>>> Both the changes and testing look good to me. Would it be better to
>>> rename MetaspaceShared::unarchive_heap_object() to
>>> MetaspaceShared::materialize_archived_object() to reflect the API
>>> in G1CollectedHeap? The use of ‘materialize’ in the GC API looks
>>> very good. Thank you for continuing improving the GC underlying
>>> support!
>> 
>> I like that suggestion.  I’ll see what other folks think, but I’m
>> inclined to do it.
>> 
> 
>  looks good. Agree with the suggested name change.
> 
> Thanks,
>  Thomas

Thanks.  I’ll make the suggested name change.



More information about the hotspot-dev mailing list