RFR (also tedious) 8216167: Update include guards to reflect correct directories

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Thu Jan 10 20:43:29 UTC 2019



On 1/10/19 3:41 PM, Kim Barrett wrote:
>> On Jan 10, 2019, at 7:34 AM, coleen.phillimore at oracle.com wrote:
>>
>>
>> No takers?  This fixes the include guard to match the file name in 1540 header files.
>>
>> If you add a header file, please use the file name after src/hotspot as the include guard name from now on (exclude the VM that used to be there).
>>
>> http://cr.openjdk.java.net/~coleenp/8216167.01.diff.02
> This deals with my issue (from the #pragma once thread) about the share/include files by excluding them from the changes.
>
> The pre-existing guard name for share/include/cds.h might not be good, but that’s not a problem with this change.
>
> This has also dealt with the incorrect changes to globalDefinitions_arm.hpp that I previously mentioned.
>
> Looks like Erik has done a good job of checking your patch.  I don’t have anything to add to that.  Looks good to me.
>
>
Thanks Kim.
Coleen


More information about the hotspot-dev mailing list