Thanks for the feedback. On 12/02/16 16:05, e...@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.
