On Fri, 21 Apr 2023 21:23:57 GMT, lawrence.andrews <[email protected]> wrote:
>> Alexander Zuev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Added couple of empty lines to make code nore readable.
>
> test/jdk/java/awt/Dialog/DialogLocationTest.java line 142:
>
>> 140: int lastX, lastY;
>> 141: String message = "Failed in componentMoved() expect:[%d,%d]"
>> 142: + " reported: [%d,%d] diff [%d,%d]";
>
> Can you leave a blank line to separate the method .
Done.
> test/jdk/java/awt/Dialog/ModalDialogOnNonEdt.java line 47:
>
>> 45:
>> 46: public class ModalDialogOnNonEdt {
>> 47: public void start () {
>
> Can you leave a blank line here
Sure.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13592#discussion_r1174232601
PR Review Comment: https://git.openjdk.org/jdk/pull/13592#discussion_r1174232561
