histogram: Treat num_buckets consistently

dtnaylor · dtnaylor · commit adf765295847 · 2019-05-14T13:25:09.000-07:00
Changes the semantics of num_buckets() to match the num_buckets
constructor argument. Setting num_buckets assumed the user did not
include the +1 bucket for above_range; getting num_buckets() did include
the above_range bucket.
diff --git a/core/utils/histogram.h b/core/utils/histogram.h
@@ -175,7 +175,7 @@ class Histogram {
     return ret;
   }
 
-  size_t num_buckets() const { return buckets_.size(); }
+  size_t num_buckets() const { return buckets_.size() - 1; }
   T bucket_width() const { return bucket_width_; }
 
   size_t max_num_buckets() const {

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ class Histogram {`
`175`	`175`	`return ret;`
`176`	`176`	`}`
`177`	`177`
`178`		`- size_t num_buckets() const { return buckets_.size(); }`
	`178`	`+ size_t num_buckets() const { return buckets_.size() - 1; }`
`179`	`179`	`T bucket_width() const { return bucket_width_; }`
`180`	`180`
`181`	`181`	`size_t max_num_buckets() const {`