On Wed, 4 Oct 2023 17:00:42 GMT, Harshitha Onkar <[email protected]> wrote:
>> Ravi Gupta has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8316947: Review comments solved
>
> test/jdk/java/awt/event/MouseEvent/MouseEnterExitTest.java line 66:
>
>> 64: public void mouseEntered(MouseEvent e) {
>> 65: System.out
>> 66: .println("MouseEntered " +
>> e.getSource().getClass().getName());
>
> It is a bit odd to have a line break after sys.out. It looks better this way.
>
>
> System.out.println("MouseEntered "
> + e.getSource().getClass().getName());
I would add the line break after "MouseEntered/Exited" as above, looks cleaner
and preserves line length.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15961#discussion_r1348985296
