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