RFR: 8153967: Remove top.hpp

Stefan Karlsson stefan.karlsson at oracle.com
Mon Apr 11 16:03:13 UTC 2016


Thanks, Coleen.

StefanK

On 2016-04-11 17:55, Coleen Phillimore wrote:
>
> 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