On Mon, 22 Aug 2022 00:57:28 GMT, SWinxy <[email protected]> wrote: >> ScientificWare has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Removes whitespace. >> >> Remove a whitespace at the end of the file > > I mean I think I would *prefer* the switch over a map (it looks nicer to me). > My crude tests showed that the switch is indeed slower, breaking my > conception that switch statements are the peak of performance. Other than > [this request](#issuecomment-1216011827), I have no further comments.
> Could we consider once more times @SWinxy 's Aug 16 comment about color > instance ? Are you talking about [keeping the old behavior](https://github.com/openjdk/jdk/pull/9825#issuecomment-1216011827)? If so, your proposed solution looks to work (also reminder that `.contains(Object)` exists). ------------- PR: https://git.openjdk.org/jdk/pull/9825
