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