RFR: 8308286 Fix clang warnings in linux code [v3]
Weijun Wang
weijun at openjdk.org
Tue Jun 6 17:50:00 UTC 2023
On Tue, 6 Jun 2023 17:32:35 GMT, Artem Semenov <asemenov at openjdk.org> wrote:
>> I didn't ask to revert the change. It's `s/TARGET_OS_MAC/defined(__APPLE__)/`.
>
> This is rarely used in the code and is not the essence of the current changes.
> If you introduce such changes, then throughout the code.
> Moreover, this can lead to problems, such as, for example, here: https://bugs.openjdk.org/browse/JDK-8309225
I'm not a clang expect, I was just asking if modifying the current `#if TARGET_OS_MAC` check into `#if defined(__APPLE__)` is also a solution. The comment on lines 46-47 says the condition was copied from a macOS SDK file and that's what the file is using now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14033#discussion_r1220071779
More information about the core-libs-dev
mailing list