RFR (XS) 8143615 compactHashtable.hpp includes oop.inline.hpp file
David Holmes
david.holmes at oracle.com
Thu Dec 3 04:49:27 UTC 2015
Hi Ioi,
On 3/12/2015 12: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
Looks okay but in compactHashtable.hpp:
#include "memory/allocation.inline.hpp"
This also needs removing - right?
Thanks,
David
> Tests:
>
> JPRT
>
> Thanks
> - Ioi
More information about the hotspot-dev
mailing list