RFR: 6434110: Color constructor parameter name is misleading [v3]

Phil Race prr at openjdk.org
Thu Feb 19 20:11:40 UTC 2026


On Tue, 17 Feb 2026 20:32:54 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

>> Small cleanup: the parameter names in the Color(int, boolean) constructor have been renamed:
>>  - rgba -> argb, since this is the format described in the spec
>>  - hasalpha -> hasAlpha, to follow common naming conventions
>> 
>> Also adds a basic test for the constructor.
>
> Sergey Bylokhov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   lists via "<ul>" is added

Marked as reviewed by prr (Reviewer).

src/java.desktop/share/classes/java/awt/Color.java line 408:

> 406:      *   <li>the red component in bits 16-23,
> 407:      *   <li>the green component in bits 8-15, and
> 408:      *   <li>the blue component in bits 0-7.

The list is cleaner.

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

PR Review: https://git.openjdk.org/jdk/pull/29734#pullrequestreview-3828251315
PR Review Comment: https://git.openjdk.org/jdk/pull/29734#discussion_r2829918733


More information about the client-libs-dev mailing list