> The current implementation of `Renderer.getPeerInstance()` looks up mappings > by concatenating strings to form a combined key: > > peer = peerCache.get(name + "_" + unrollCount); > > > This can be improved by not using strings to store the combined key, but > using a simple `PeerCacheKey` class instead. > The `Renderer` class then uses a single reusable `PeerCacheKey` object to > look up mappings, making the lookup allocation-free.
Michael Strauß has updated the pull request incrementally with two additional commits since the last revision: - HashMap.newHashMap - don't use Collections.synchronizedMap ------------- Changes: - all: https://git.openjdk.org/jfx/pull/2091/files - new: https://git.openjdk.org/jfx/pull/2091/files/c14ac681..9a059bb8 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=2091&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2091&range=01-02 Stats: 67 lines in 4 files changed: 39 ins; 7 del; 21 mod Patch: https://git.openjdk.org/jfx/pull/2091.diff Fetch: git fetch https://git.openjdk.org/jfx.git pull/2091/head:pull/2091 PR: https://git.openjdk.org/jfx/pull/2091
