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

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Fri Jan 4 23:19:15 UTC 2019



On 1/4/19 4:52 PM, Kim Barrett wrote:
>> On Jan 4, 2019, at 10:36 AM, coleen.phillimore at oracle.com wrote:
>>
>> Summary: Use script and some manual fixup to fix directores names in include guards.
>>
>> Makes include guards match the current directory rooted at src/hotspot (removes VM_ in most cases).
>>
>> This should be low risk.  Tested with mach5 tier1 and tier2.
>>
>> https://bugs.openjdk.java.net/browse/JDK-8216167
>> http://cr.openjdk.java.net/~coleenp/8216167.01.diff
>>
>> I didn't generate a webrev as a space concern for cr.openjdk.java.net and nobody should click on it.  Script is posted in bug.  Will update and check copyright headers with hg commit.
>>
>> Thanks,
>> Coleen
> There are incorrect changes in src/hotspot/cpu/arm/globalDefinitions_arm.hpp

Thank you for finding this.
> The script is not being careful to *only* modify #include guards.  I didn’t look for other similar problems.
> (This is an example of why I suggested rolling your own script might not actually be easier than using
> the guardonce utilities.)
>
I looked through again and didn't see any other problems.  I'm not 
planning on productizing my script, which admittedly is too simple for 
the general job.  But I still found it useful and entertaining (to me) 
for this particular task.

http://cr.openjdk.java.net/~coleenp/8216167.01.diff.02

Thanks,
Coleen


More information about the hotspot-dev mailing list