RFR: 8153742: Remove unnecessary thread.inline.hpp includes

Stefan Karlsson stefan.karlsson at oracle.com
Thu Apr 7 17:11:26 UTC 2016


Thanks, Coleen!

StefanK

On 2016-04-07 19:08, Coleen Phillimore wrote:
>
> Looks good. Thanks for doing this!
> Coleen
>
> On 4/7/16 12:41 PM, Stefan Karlsson wrote:
>> Hi all,
>>
>> Please review this tiny patch to remove includes of thread.inline.hpp 
>> from some .hpp files.
>>
>> http://cr.openjdk.java.net/~stefank/8153742/webrev.01/
>> https://bugs.openjdk.java.net/browse/JDK-8153742
>>
>> I'm especially interested in getting rid of the include inside 
>> resourceArea.hpp, since that will allow me to setup a ResourceMark in 
>> ostream.hpp without leaking inline.hpp files.
>>
>> I've verified that none of the changed files use any of the functions 
>> in thread.inline.hpp.
>>
>> Thanks,
>> StefanK
>



More information about the hotspot-dev mailing list