Hi Mike,

On 15/11/2012 3:49 AM, Mike Duigou wrote:
Hello all;

A small but useful performance fix for sub-maps of TreeMap:

http://cr.openjdk.java.net/~mduigou/7175464/0/webrev/

The entrySetView was not being cached.

Seems a bug that entrySetView was never being set, but given that it wasn't this may now introduce an incompatability may it not? Even if the spec permits caching for how long have we always returned a new instance?

David

There is no unit test because either implementation is permissible under the 
specification. The fix only has the effect of improving performance and 
reducing duplicate objects.

Mike

Reply via email to