On Tue, 22 Sep 2026 16:57:22 GMT, Phil Race <[email protected]> wrote:
>> Prasanta Sadhukhan has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Test update
>
> test/jdk/javax/print/GetUIClassNameForRole.java line 54:
>
>> 52: DOCUMENT_PROPERTIES_ROLE);
>> 53:
>> 54: if (names == null || names.length != 1) {
>
> Returning null is perfectly legal. Returning more than one is also perfectly
> legal.
> I think this test should be re-worked to test the actual bug which was an
> AIOOBE.
> It doesn't have to be windows-specific.
> It just needs to call the API on the factory, if any and check it doesn't get
> the AIIOBE.
ok..test updated..
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33009#discussion_r4078847964
