On Mon, 24 Aug 2026 15:07:44 GMT, Coleen Phillimore <[email protected]> wrote:

>> Shiv Shah has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Address review comments, use Log and single line calls
>
> test/hotspot/jtreg/vmTestbase/nsk/jdi/Accessible/isPrivate/isPrivate001a.java 
> line 83:
> 
>> 81: 
>> 82:     private static void display(String message) {
>> 83:         System.err.println(message);
> 
> Can you change these to log.display? and not have a display method here at 
> all?

Done, dropped the local method everywhere and switched to log.display backed by 
new Log(System.err), same stream as before. the split calls got joined as part 
of the rename

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32375#discussion_r3848277880

Reply via email to