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

Reply via email to