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

[Harshitha Onkar]

On Mon, 24 Feb 2025 18:59:34 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   javadoc update
>
> src/java.desktop/share/classes/java/awt/color/ICC_Profile.java line 116:
> 
>> 114:      * BuiltInProfile.
>> 115:      */
>> 116:     private boolean isBuiltIn = false;
> 
> Should the field be named `builtIn` instead? `isBuiltIn` starts with a verb which is used for naming methods.

IIRC we usually have `is` or `has` prefixed to boolean vars as per naming convention?

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

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