Request for review (s) - NPG: Replace the ChunkList implementation with class FreeList<Metachunk>

[Jon Masamitsu]

Thomas,

Thanks for look the review.

On 4/4/13 1:33 AM, Thomas Schatzl wrote:
> Hi,
>
> On Tue, 2013-04-02 at 16:42 +0200, Mikael Gerdin wrote:
>> Jon,
>>
>> On 2013-04-02 00:13, Jon Masamitsu wrote:
>>> For the perm gen removal project the ChunkList class
>>> which implemented a simple freelist of Metachunk's was added.
>>> This change replaces ChunkList with FreeList<Metachunk>.
>>>
>>> Changes include
>>>
>>> - Deletion of ChunkList
>>> - Code to initialize the freelists in the FreeList style
>>> - Replacement of call to ChunkList methods with the
>>> equivalent calls to FreeList methods.
>>>
>>> http://cr.openjdk.java.net/~jmasa/8011173
>> This looks like a good cleanup step.
> Same here.
>
>> The name FreeList is kind of a misnomer since it's actually just the
>> head of a generic linked list.
> Noticed the same about the naming. Other than that, seems fine.

Yes, wrong name.  I filed 8011265 to fix it.

Jon
>
> Thomas
>
>
>