In reviewing the code for bug 436 (http://issues.apache.org/jira/browse/LUCENE-436)
Why are we using a ThreadLocal for the enumeration at all? Since terms(), and terms(Term t) return new instances anyway, why not just have them clone the needed data structures? Seems like the code could be much simpler and perform just as well. What am I missing?
