On Mon, 6 Sep 2021 13:30:46 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Jose Pereda has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Use color to int32 converter instead of hash
>
>> The other option, as you mention, is finding out if the 32-bit int has alpha 
>> 0 or 1, which can be done storing only the int value, not the color, so this 
>> looks cleaner, we don't really need to hold a reference to the Color after 
>> all
> 
> Yes, I like this better, too.
> 
> I also agree with the point about including `rgba` in the name. It is also 
> meaningful to keep `int` in the name. We would typically call this format 
> `IntRgba`.

@kevinrushforth It should be fixed now, the test should pass on the three 
platforms.

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

PR: https://git.openjdk.java.net/jfx/pull/563

Reply via email to