> Do we use StatHist::operator "=="? If not, I suggest not adding it > because the added code may return false for histograms that are > essentially the same for many practical purposes. In general, what > histograms are considered the same depends on the caller use of > statistics so if we do not have any callers, let's not add this code. > > No need to repost the patch with this minor change, of course.
It is only used in the unit tests. I can remove that if you feel that'd be better. I don't know how to meaningfully perform unit testing without adding to the class' interface, as it has virtually no useable accessors. -- /kinkie
