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

Stefan Karlsson stefan.karlsson at oracle.com
Thu Mar 8 09:36:08 UTC 2018

Further testing on this with other build targets show that I need to add 
handles.inline.hpp includes to cpp files that used to get it included 
from reflectionUtils.hpp. I'll post a new webrev when I've run all build 


On 2018-03-08 10:14, 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