Made StatHist::bins private

diff --git a/src/StatHist.cc b/src/StatHist.cc
index 85f28bd3dc3dee134b4109e1cba3cc1e185818c6..2ec43bcf547c6b253fc5b4b9cae6856ee50b0648 100644 (file)
--- a/src/StatHist.cc
+++ b/src/StatHist.cc
@@ -151,6 +151,12 @@ statHistDeltaMedian(const StatHist & A, const StatHist & B)
 
 double
 statHistDeltaPctile(const StatHist & A, const StatHist & B, double pctile)
+{
+    return A.deltaPctile(B,pctile);
+}
+
+double
+StatHist::deltaPctile(const StatHist & B, double pctile) const
 {
     int i;
     int s1 = 0;
@@ -158,25 +164,25 @@ statHistDeltaPctile(const StatHist & A, const StatHist & B, double pctile)
     int a = 0;
     int b = 0;
     int I = 0;
-    int J = A.capacity;
+    int J = capacity;
     int K;
     double f;
 
-    assert(A.capacity == B.capacity);
+    assert(capacity == B.capacity);
 
-    int *D = (int *)xcalloc(A.capacity, sizeof(int));
+    int *D = (int *)xcalloc(capacity, sizeof(int));
 
-    for (i = 0; i < A.capacity; ++i) {
-        D[i] = B.bins[i] - A.bins[i];
+    for (i = 0; i < capacity; ++i) {
+        D[i] = B.bins[i] - bins[i];
         assert(D[i] >= 0);
     }
 
-    for (i = 0; i < A.capacity; ++i)
+    for (i = 0; i < capacity; ++i)
         s1 += D[i];
 
     h = int(s1 * pctile);
 
-    for (i = 0; i < A.capacity; ++i) {
+    for (i = 0; i < capacity; ++i) {
         J = i;
         b += D[J];
 
@@ -206,7 +212,7 @@ statHistDeltaPctile(const StatHist & A, const StatHist & B, double pctile)
 
     K = (int) floor(f * (double) (J - I) + I);
 
-    return A.val(K);
+    return val(K);
 }
 
 static void

diff --git a/src/StatHist.h b/src/StatHist.h
index 8afd5b605aa3f98763f2ee87ee6a87ce1ac41c87..f80f9376a1fdd736343b6a3f3a3481fe37819a4c 100644 (file)
--- a/src/StatHist.h
+++ b/src/StatHist.h
@@ -42,7 +42,6 @@
 class StatHist {
 public:
     void clear();
-    int *bins;
     int capacity;
     double min;
     double max;
@@ -53,16 +52,18 @@ public:
     double val(int bin) const; //todo: make private
     void count(double val);
     StatHist &operator=(const StatHist &);
-    StatHist() : bins(NULL), capacity(0), min(0), max(0), scale(1.0),
-            val_in(NULL), val_out(NULL) {};
+    StatHist() : scale(1.0) {};
     StatHist(const StatHist&);
     void dump(StoreEntry *sentry, StatHistBinDumper * bd) const;
     void logInit(int capacity, double min, double max);
     void enumInit(int last_enum);
     void intInit(int n);
     void init(int capacity, hbase_f * val_in, hbase_f * val_out, double min, double max);
+    double deltaPctile(const StatHist & B, double pctile);
 private:
     int findBin(double v);
+    int *bins;
+
 };
 
 /* StatHist */