RFR: 8037952: Remove code duplication in Metaspace::deallocate

[Coleen Phillimore]

So Stefan reminded me that class file parsing deallocates metadata 
outside a safepoint.  I thought it was some crazy GC doing class 
unloading.  So this is fine, never mind.
Coleen

On 3/20/14 12:00 PM, Coleen Phillimore wrote:
>
> I thought we always did this at safepoint and the 'else' code was 
> duplicate, but that's not the case if the current thread isn't the VM 
> thread?   Please add a comment about this!
>
> Thanks,
> Coleen
>
> On 3/20/14 10:12 AM, Stefan Karlsson wrote:
>> Please, review this cleanup patch to remove some unnecessary code 
>> duplication:
>>
>> http://cr.openjdk.java.net/~stefank/8037952/webrev.00/
>>
>> thanks,
>> StefanK
>