: Here's the changes I'll commit shortly unless there are objections : (sorry about the false-changes... I think it must be trailing
one thing confuses me: why the boolean params for missing/zero in the method declarations for getFieldCacheCounts and getFieldTermEnumCounts ? ... for getFieldTermEnumCounts i was pulling that info from the protected member variable "params" to keep the method signatures simple for when the day comes that we want the behavior to be driven by additional params (i expect there will be custom subclasses to SimpleFacets in the future reusing these methods, and i'd hate to keep adding method params that make it hard for those subclasses) -Hoss
