On Wed, 13 Mar 2024 17:29:46 GMT, Alexey Ivanov <[email protected]> wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> minor formatting
>
> test/jdk/java/awt/Frame/GetBoundsResizeTest.java line 75:
>
>> 73: EventQueue.invokeAndWait(() ->
>> 74: textArea.append("Original Frame.getBounds() = %s\n"
>> 75: .formatted(frame.getBounds())));
>
> Suggestion:
>
> textArea.append("Original Frame.getBounds() = %s\n"
> .formatted(frame.getBounds())));
>
> I believe @honkar-jdk made you update formatting… whereas the previous
> version looked right. Since `.formatted` is called on the string literal, it
> should be aligned to the start of the literal, to the opening quote. With
> such formatting, it could be misinterpreted to be called on the result of
> `textArea.append` which doesn't return anything.
The same applies to the following event handler.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18250#discussion_r1523730047
