RFR: 8286270: [java.desktop] Replace color search in XColors with a switch statement [v3]
Phil Race
prr at openjdk.org
Thu Jul 14 00:26:09 UTC 2022
On Fri, 20 May 2022 00:53:27 GMT, SWinxy <duke at openjdk.org> wrote:
>> The current implementation creates ~750 new objects on load, and uses a binary array search with a Comparable implementation. This implementation is probably bad. Instead, we can use the enhanced switch feature to possibly save time, resources, and code size. The new implementation means that the array doesn't have to be sorted alphanumerically necessarily, and it creates the colors on demand.
>
> SWinxy has updated the pull request incrementally with one additional commit since the last revision:
>
> Accidental mergers.
>
> I created a quick check to see if any other colors were different (only these three). For some odd reason "yellowgreen" and "yellow green" are different.
> I've hesitated asking for help, but the builds aren't working and I can't tell _why_.
the GHA builds are passing so far as I can tell.
-------------
PR: https://git.openjdk.org/jdk/pull/7096
More information about the client-libs-dev
mailing list