On Fri, 5 Jul 2024 07:51:08 GMT, Julian Waters <jwat...@openjdk.org> wrote:

>> Please review this change to some jvmti tests, which were calling fflush with
>> an argument of 0. Most of these are in C++ code, where we change them to use
>> nullptr as the argument.  A couple are in C, where we change them to use 
>> NULL.
>> This removes a bunch of -Wzero-as-null-pointer-constant when building with
>> that enabled.
>> 
>> Testing: mach5 tier1
>
> Marked as reviewed by jwaters (Committer).

Thanks for reviews @TheShermanTanker and @coleenp

-------------

PR Comment: https://git.openjdk.org/jdk/pull/20032#issuecomment-2217703027

Reply via email to