RFR (XS) 8143615 compactHashtable.hpp includes oop.inline.hpp file

Jiangli Zhou jiangli.zhou at oracle.com
Thu Dec 3 04:25:52 UTC 2015


+1

Thanks,
Jiangli

> On Dec 2, 2015, at 6:35 PM, Coleen Phillimore <coleen.phillimore at oracle.com> wrote:
> 
> 
> Looks good.  Thank you for fixing this.
> Coleen
> 
> On 12/2/15 9:32 PM, Ioi Lam wrote:
>> Please review a very small fix:
>> 
>> http://cr.openjdk.java.net/~iklam/8143615-compactHashtable-inline-hpp/
>> 
>> Bug: compactHashtable.hpp includes oop.inline.hpp file
>> 
>>    https://bugs.openjdk.java.net/browse/JDK-8143615
>> 
>> Summary of fix:
>> 
>>    + deleted the include of oop.inline.hpp from compactHashtable.hpp
>>    + moved all inlined functions that require oop.inline.hpp into a
>>      new file, compactHashtable.inline.hpp
>> 
>> Tests:
>> 
>>    JPRT
>> 
>> Thanks
>> - Ioi
> 



More information about the hotspot-dev mailing list