On Fri, 26 Sep 2025 07:53:44 GMT, Prasanta Sadhukhan <[email protected]> 
wrote:

>> Ravi-Patel8 has updated the pull request incrementally with two additional 
>> commits since the last revision:
>> 
>>  - Updated Comments
>>    
>>    Signed-off-by: Ravi.Patel8 <[email protected]>
>>  - Add catch block to log exception details for troubleshooting printer
>>    command
>>    
>>    Signed-off-by: Ravi.Patel8 <[email protected]>
>
> 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

Reply via email to