RFR: 8314114: Fix -Wconversion warnings in os code, primarily linux [v2]
Thomas Stuefe
stuefe at openjdk.org
Sat Aug 12 06:58:58 UTC 2023
On Fri, 11 Aug 2023 11:58:28 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
>> src/hotspot/os/aix/attachListener_aix.cpp line 296:
>>
>>> 294: // hang in the clean-up when shutting down.
>>> 295: n = read(s, buf+off, left);
>>> 296: assert(n <= checked_cast<ssize_t>(left), "buffer was too small, impossible!");
>>
>> why ssize_t here?
>
> This gets a sign comparison warning without the cast, which thankfully we do enable.
I'm curious, who is warning here? Is this a static code analysis? Since I don't think Oracle builds AIX, right?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15229#discussion_r1292117246
More information about the hotspot-dev
mailing list