RFR: 8368606: Printer lookup returns empty on AIX platform due to uninitialized results list [v2]
Phil Race
prr at openjdk.org
Fri Sep 26 20:49:21 UTC 2025
On Fri, 26 Sep 2025 07:53:44 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> Ravi-Patel8 has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Updated Comments
>>
>> Signed-off-by: Ravi.Patel8 <Ravi.Patel8 at ibm.com>
>> - Add catch block to log exception details for troubleshooting printer
>> command
>>
>> Signed-off-by: Ravi.Patel8 <Ravi.Patel8 at ibm.com>
>
> src/java.desktop/unix/classes/sun/print/PrintServiceLookupProvider.java line 887:
>
>> 885: } catch (Exception e) {
>> 886: // Print exception for tracking printer command errors
>> 887: System.err.println(e);
>
> Dont think we use println directly...you can use `IPPPrintService.debug_println` instead in my opinion
agree
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27482#discussion_r2383466948
More information about the client-libs-dev
mailing list