On Wed, 13 Mar 2024 17:29:46 GMT, Alexey Ivanov <aiva...@openjdk.org> 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