RFR: 8366208: Unexpected exception in sun.java2d.cmm.lcms.LCMSImageLayout [v2]
Alexey Ivanov
aivanov at openjdk.org
Fri Aug 29 19:23:43 UTC 2025
On Fri, 29 Aug 2025 19:16:09 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
>> test/jdk/sun/java2d/cmm/ColorConvertOp/FilterSemiCustomImages.java line 87:
>>
>>> 85: private static void test(int fromIndex, int toIndex, int type)
>>> 86: throws Exception
>>> 87: {
>>
>> Suggestion:
>>
>> private static void test(int fromIndex, int toIndex, int type)
>> throws Exception {
>>
>> Java code style prefers the opening brace is on the same line instead of its own line.
>
> the java2d code uses the style that the "{" is on new line if the line was wrapped. That was forced by Jim, and I still follow that.
Okay. How much common is it for the newly added code? I haven't come across it frequently recently.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26994#discussion_r2310945437
More information about the client-libs-dev
mailing list