RFR: 8199319: Remove handles.inline.hpp include from reflectionUtils.hpp

David Holmes david.holmes at oracle.com
Thu Mar 8 10:02:45 UTC 2018


Looks fine.

Thanks,
David

On 8/03/2018 7:14 PM, Stefan Karlsson wrote:
> Hi all,
> 
> Please review this tiny patch to remove an unnecessary include of 
> handles.inline.hpp. This is a step towards getting cleaner include 
> dependencies.
> 
> http://cr.openjdk.java.net/~stefank/8199319/webrev.01/
> 
> I compiled this with gcc -E to look for indirect includes of .inline.hpp 
> files. While doing this I found that this file couldn't be compiled 
> stand-alone. So, I also added the missing growableArray.hpp include.
> 
> Thanks,
> StefanK


More information about the hotspot-runtime-dev mailing list