On Mon, 9 Mar 2026 20:05:14 GMT, Sergey Bylokhov <[email protected]> wrote:

>> test/jdk/sun/java2d/OpenGL/FlipCoexistTest.java line 110:
>> 
>>> 108:     private static void check(Robot robot, Frame frame, int x, int y, 
>>> Color exp,
>>> 109:                               String desc)
>>> 110:     {
>> 
>> Suggestion:
>> 
>>     private static void check(Robot robot, Frame frame, int x, int y, Color 
>> exp,
>>                               String desc) {
>> 
>> Is there a specific reason of putting the opening brace on its own line?
>
> That is the only method that splits the list of arguments.

Even in that case, the brace follows the closing parenthesis after parameter 
declaration.

See 
https://www.oracle.com/java/technologies/javase/codeconventions-indentation.html#248


//CONVENTIONAL INDENTATION
someMethod(int anArg, Object anotherArg, String yetAnotherArg,
           Object andStillAnother) {
    ...
}

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

PR Review Comment: https://git.openjdk.org/jdk/pull/29886#discussion_r2907704692

Reply via email to