On Fri, 6 Nov 2026 07:34:49 GMT, Jeremy Wood <[email protected]> wrote:

>> This resolves the NPE reported in 8395930, and it enhances the preexisting 
>> test for JDK-8381236 to cover this case.
>> 
>> The failing use case involves an Accessible object where 
>> `axObj.getAccessibleContext() == null`. Having a `null` AccessibleContext is 
>> an atypical case, but it is explicitly identified/allowed in the contract 
>> for `Accessible.getAccessibleContext()`.
>> 
>> Prior to [8381236 
>> changes](https://github.com/openjdk/jdk/pull/30578/changes), calling 
>> CAccessible.getCAccessible(ax) would also fail with a NPE. So in this PR I'm 
>> proposing preserving that behavior (and documenting it), and moving the null 
>> check to 8381236's new `disposeRecursively` method.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Jeremy Wood has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   8395930: cleaning up javadoc & formatting

Marked as reviewed by kizune (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/33657#pullrequestreview-5810118627

Reply via email to