On Thu, 20 Mar 2025 19:56:45 GMT, Sergey Bylokhov <[email protected]> wrote:
>> Harshitha Onkar has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> redudant stmt removed
>
> test/jdk/java/awt/color/ICC_Profile/BuiltInProfileCheck.java line 50:
>
>> 48:
>> 49: private static void updateProfile(boolean isBuiltIn) {
>> 50: ICC_Profile builtInProfile =
>> ICC_Profile.getInstance(ColorSpace.CS_sRGB);
>
> It would be good to check all types of BuiltIn profiles to prove that it
> works in case we decide to implement/change some them differently(w/o cache
> for example).
Updated
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23606#discussion_r2006671703
