> 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 one additional commit since the last revision: review comment ------------- Changes: - all: https://git.openjdk.org/jfx/pull/2091/files - new: https://git.openjdk.org/jfx/pull/2091/files/9a059bb8..d48d08b5 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx&pr=2091&range=03 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2091&range=02-03 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 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
