RFR: 8327173: HotSpot Style Guide needs update regarding nullptr vs NULL [v2]
    Kim Barrett 
    kbarrett at openjdk.org
       
    Tue Mar  5 07:12:09 UTC 2024
    
    
  
On Mon, 4 Mar 2024 09:52:15 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Kim Barrett has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   respond to shipilev comments
>
> doc/hotspot-style.md line 738:
> 
>> 736: expressions with value zero.  C++14 replaced that with integer literals with
>> 737: value zero.  Some compilers continue to treat a zero-valued integral constant
>> 738: expressions as a _null pointer constant_ even in C++14 mode.
> 
> It is not very clear why should this concern any Hotspot developers?
If one is familiar with C++14 (or later) or were to look up _null pointer
constant_ in the C++14 standard, one might wonder why this style guide
admonishes against using non-literal constant expressions for such. But I've
rewritten the part about 0 as a pointer to (I think) make things clearer.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18101#discussion_r1512225238
    
    
More information about the build-dev
mailing list