RFR: 8309558: Create implementation of NSAccessibilityCheckBox protocol [v2]
Kevin Rushforth
kcr at openjdk.org
Mon Aug 7 14:13:40 UTC 2023
On Sun, 6 Aug 2023 07:45:54 GMT, Ambarish Rapte <arapte at openjdk.org> wrote:
>> Alexander Zuev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Add newline at the end of the header files.
>
> modules/javafx.graphics/src/main/native-glass/mac/a11y/AccessibleBase.m line 174:
>
>> 172: jobject jresult = NULL;
>> 173: GET_MAIN_JENV;
>> 174: if (env == NULL) return NULL;
>
> This generates a warning:
> `warning: incompatible pointer to integer conversion returning 'void *' from a function with result type 'BOOL' (aka 'signed char') [-Wint-conversion]`
Good catch. This should probably be changed to: `return NO;`
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1184#discussion_r1285921001
More information about the openjfx-dev
mailing list