On Fri, 12 Jan 2024 22:53:08 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> Sergey Bylokhov has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Arrays.toString
>
> test/jdk/java/awt/color/ICC_ColorSpace/SimpleSRGBToFromCIEXYZ.java line 46:
>
>> 44: System.err.println("XYZ color:\t\t" +
>> Arrays.toString(xyz));
>> 45: System.err.println("Actual color:\t" +
>> Arrays.toString(inv));
>> 46: throw new java.lang.Error("Wrong color");
>
> Is the full class name required? All classes from `java.lang` are imported,
> aren't they?
for test execution it does not matter but when opened in ide it conflicts with
some other Error classes from the tests.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17388#discussion_r1451033233
