RFR: JDK-8324682 Remove redefinition of NULL for XLC compiler [v5]
Christoph Langer
clanger at openjdk.org
Thu Mar 7 10:59:55 UTC 2024
On Thu, 7 Mar 2024 08:13:52 GMT, Suchismith Roy <sroy at openjdk.org> wrote:
>> Remove redefinition of NULL for XLC compiler
>> In globalDefinitions_xlc.hpp there is a redefinition of NULL to work around an issue with one of the AIX headers (<dirent.h>).
>> Once all uses of NULL in HotSpot have been replaced with nullptr, there is no longer any need for this, and it can be removed.
>>
>>
>> JBS ISSUE : [JDK-8324682](https://bugs.openjdk.org/browse/JDK-8324682)
>
> Suchismith Roy has updated the pull request incrementally with one additional commit since the last revision:
>
> update comment and copyright
Changes requested by clanger (Reviewer).
src/hotspot/share/utilities/globalDefinitions_xlc.hpp line 2:
> 1: /*
> 2: * Copyright (c) 1998, 2023, Oracle and/or its affiliates. All rights reserved.
You need to update the year for the Oracle copyright. Adding additional copyright lines, such as IBM, is usually only done when you substantially change a file, i.e. add larger non-trivial portions of code.
-------------
PR Review: https://git.openjdk.org/jdk/pull/18064#pullrequestreview-1922127498
PR Review Comment: https://git.openjdk.org/jdk/pull/18064#discussion_r1515961111
More information about the hotspot-dev
mailing list