RFR [9 ] 8149656: Examine usages of sun.misc.LRUCache
Chris Hegarty
chris.hegarty at oracle.com
Fri Feb 12 16:26:30 UTC 2016
Thanks for the feedback.
On 12/02/16 16:05, ecki at zusammenkunft.net wrote:
> Since it is only implemented with one anonymous class you could also think about making it a concrete PatternLRUCache
On 12/02/16 15:59, Claes Redestad wrote:
> +1
>
> nits: I guess the class should be made static if nested here, or even
> made concrete to get rid of the anonymous class - that depends on if we
> think we'll ever reuse this utility for other things.
With all these changes, my opening position is to keep things
the same. Accepting the move, then there is of course much
refactoring/cleanup that can be done. Here is the next
iteration, with your suggestions.
http://cr.openjdk.java.net/~chegar/8149656.01/
-Chris.
More information about the core-libs-dev
mailing list