RFR (tedious) 8216022: Use #pragma once

Per Liden per.liden at oracle.com
Mon Jan 7 08:40:04 UTC 2019


Hi Coleen,

On 1/3/19 3:31 AM, coleen.phillimore at oracle.com wrote:
> 
> Here is the webrev and bug link.
> 
> open webrev at http://cr.openjdk.java.net/~coleenp/8216022.01/webrev

Looks like your script is now leaving an extra empty line at the end of 
all files, which wasn't there before. Fox example:

--- old/src/hotspot/share/gc/z/zAddress.hpp	2019-01-02 
16:41:04.209075410 -0500
+++ new/src/hotspot/share/gc/z/zAddress.hpp	2019-01-02 
16:41:03.957075419 -0500
@@ -64,4 +63,3 @@
    static void flip_to_remapped();
  };

-#endif // SHARE_GC_Z_ZADDRESS_HPP

Should be:

--- old/src/hotspot/share/gc/z/zAddress.hpp	2019-01-02 
16:41:04.209075410 -0500
+++ new/src/hotspot/share/gc/z/zAddress.hpp	2019-01-02 
16:41:03.957075419 -0500
@@ -64,4 +63,3 @@
    static void flip_to_remapped();
  };
-
-#endif // SHARE_GC_Z_ZADDRESS_HPP


Could you please fix that?

Thanks!
Per

> bug link https://bugs.openjdk.java.net/browse/JDK-8216022
> 
> On 1/2/19 9:16 PM, coleen.phillimore at oracle.com wrote:
>> Summary: change include guards to #pragma once, except in generated 
>> header files.
>>
>> Tested with mach5 for linux-x64{-debug}, solaris-sparc, macosx-x64, 
>> windows-x64, built aarch64 with cross compiler, and zero.
>>
>> Ran tier1 and 2 tests.
>>
>> The webrev is huge but there are only 3 lines changed in each header 
>> file.  So click on the patch.
>>
>> I'll update the copyright headers with a script with the commit. Also, 
>> will do this after the shenandoah copyright headers are fixed.
>>
>> Adrian: I included you to check your platforms.
>>
>> Happy New Year!
>> Coleen
> 


More information about the hotspot-dev mailing list