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