> 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:

  add test

-------------

Changes:
  - all: https://git.openjdk.org/jfx/pull/2091/files
  - new: https://git.openjdk.org/jfx/pull/2091/files/d26b8d25..c14ac681

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx&pr=2091&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx&pr=2091&range=00-01

  Stats: 91 lines in 2 files changed: 91 ins; 0 del; 0 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

Reply via email to