RFR: JDK-8346465 : Add a check in setData() to restrict the update of Built-In ICC_Profiles [v13]

Harshitha Onkar honkar at openjdk.org
Tue Mar 11 17:47:55 UTC 2025


On Mon, 10 Mar 2025 23:33:39 GMT, Harshitha Onkar <honkar at openjdk.org> wrote:

>> test/jdk/java/awt/color/ICC_Profile/BuiltInProfileCheck.java line 71:
>> 
>>> 69:             } catch (IllegalArgumentException iae) {
>>> 70:                 if (!iae.getMessage().equals(EXCEPTION_MSG)) {
>>> 71:                     throw new RuntimeException("Test Failed! IAE with exception msg \""
>> 
>> I don't think you need to - or should - expect a specific exception message.
>> Obviously you can update the test if the text changes, but I just don't think this is necessary.
>
> Updated.

Reverted based on @aivanov-jdk  suggestion - https://github.com/openjdk/jdk/pull/23606#discussion_r1989287031

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23606#discussion_r1989837589


More information about the client-libs-dev mailing list