On 01/14/2015 10:50 AM, Peter Levart wrote:
Hi,
Loosely related to this topic, there is some local variable caching of
comparableClassFor() result already being performed inside iterative/recursive
methods of TreeNode, but this caching is just positive caching, meaning that
null return is not cached. For keys that happen to implement Comparable, but are
not simple-self-comparable (their class is not of the form: class C implements
Comparable<C>), reflection is invoked each time comparison of the search key
with Node key is attempted. This can be improved:
http://cr.openjdk.java.net/~plevart/jdk9-dev/HM.comparableClassFor/InCallNegativeCache/webrev.01/
Thanks! This seems like a definite improvement, that we ought to
put in place.
-Doug
