RFR: 8153967: Remove top.hpp

Coleen Phillimore coleen.phillimore at oracle.com
Mon Apr 11 15:55:43 UTC 2016


This looks great.  Never wanted to understand why we had top.hpp.
Thanks!
Coleen

On 4/11/16 11:11 AM, Stefan Karlsson wrote:
> Thanks, Jesper!
>
> StefanK
>
> On 2016-04-11 17:06, Jesper Wilhelmsson wrote:
>> Looks good!
>> /Jesper
>>
>> Den 11/4/16 kl. 16:16, skrev Stefan Karlsson:
>>> Hi all,
>>>
>>> Please review this patch remove the top.hpp file.
>>>
>>> http://cr.openjdk.java.net/~stefank/8153967/webrev.01
>>> https://bugs.openjdk.java.net/browse/JDK-8153967
>>>
>>> It has been replaced by explicit includes and forward declarations, 
>>> where
>>> needed. This is just a small step towards getting smaller include 
>>> dependencies.
>>>
>>> Tested with JPRT
>>>
>>> Thanks,
>>> StefanK
>



More information about the hotspot-dev mailing list