On Wed, 23 Sep 2026 20:50:15 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 44:
>
>> 42: for (PrintService service :
>> 43: PrintServiceLookup.lookupPrintServices(null, null)) {
>> 44: ServiceUIFactory factory = service.getServiceUIFactory();
>
> That should be
> if (factory != null) { ... }
It passes in CI without null check in all platforms...but anyways added null
check
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/33009#discussion_r4090573943