On Wed, 13 Jan 2021 21:24:57 GMT, Phil Race <p...@openjdk.org> wrote:
>> src/java.desktop/macosx/native/libawt_lwawt/awt/JavaComponentAccessibility.m
>> line 1456:
>>
>>> 1454:
>>> 1455: id value = nil;
>>> 1456: if ((*env)->IsInstanceOf(env, jparent, jc_Container)) {
>>
>> Not sure, but do we need to check the jparent to NULL? "A NULL object can be
>> cast to any class."
>
> Fair question but there was no NULL check there before so I assume it isn't
> expected to be NULL.
> I could add a NULL check but then we'd skip over the following code and maybe
> hide something that could be a problem. I don't mind either way.
I do not know how exactly the JNFIsInstanceOf works for NULL parameters, will
it return true or false?
-------------
PR: https://git.openjdk.java.net/jdk/pull/2066
