RFR: 6434110: Color constructor parameter name is misleading [v2]
Alexey Ivanov
aivanov at openjdk.org
Tue Feb 17 20:23:40 UTC 2026
On Tue, 17 Feb 2026 20:19:02 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
>> I understand that, but prefer >>> for alpha only. "()" is added to make operation order clear.
>
> You can still use `>>>` for other components, it will look consistent at least. Otherwise, you can't help wondering why different operators are used.
That is
int expR = (argb >>> 16) & 0xFF;
int expG = (argb >>> 8) & 0xFF;
wouldn't raise questions.
(And I was wrong in my suggestion above, you still need `&`.)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29734#discussion_r2818904872
More information about the client-libs-dev
mailing list