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