From: Noah Watkins <noahwatkins@gmail.com>
Date: Wed, 30 Oct 2013 18:58:29 +0000 (-0700)
Subject: CrushTester: remove unused private var [-Wunused-private-field]
X-Git-Tag: v0.73~39^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=cf29574203ce9a35e5aa145832bc9f1556d0f41a;p=ceph.git

CrushTester: remove unused private var [-Wunused-private-field]

Signed-off-by: Noah Watkins <noahwatkins@gmail.com>
---

diff --git a/src/crush/CrushTester.h b/src/crush/CrushTester.h
index 0cbab0fd988..7a681e253db 100644
--- a/src/crush/CrushTester.h
+++ b/src/crush/CrushTester.h
@@ -12,7 +12,6 @@
 class CrushTester {
   CrushWrapper& crush;
   ostream& err;
-  int verbose;
 
   map<int, int> device_weight;
   int min_rule, max_rule;
@@ -165,8 +164,8 @@ class CrushTester {
    void write_integer_indexed_scalar_data_string(vector<string> &dst, int index, float scalar_data);
 
 public:
-  CrushTester(CrushWrapper& c, ostream& eo, int verbosity=0)
-    : crush(c), err(eo), verbose(verbosity),
+  CrushTester(CrushWrapper& c, ostream& eo)
+    : crush(c), err(eo),
       min_rule(-1), max_rule(-1),
       min_x(-1), max_x(-1),
       min_rep(-1), max_rep(-1),
diff --git a/src/mon/OSDMonitor.cc b/src/mon/OSDMonitor.cc
index d56953ba069..c028af11e73 100644
--- a/src/mon/OSDMonitor.cc
+++ b/src/mon/OSDMonitor.cc
@@ -2923,7 +2923,7 @@ bool OSDMonitor::prepare_command(MMonCommand *m)
     // sanity check: test some inputs to make sure this map isn't totally broken
     dout(10) << " testing map" << dendl;
     stringstream ess;
-    CrushTester tester(crush, ess, 1);
+    CrushTester tester(crush, ess);
     tester.test();
     dout(10) << " result " << ess.str() << dendl;
 
diff --git a/src/tools/crushtool.cc b/src/tools/crushtool.cc
index 03c83f24156..caebf2d5620 100644
--- a/src/tools/crushtool.cc
+++ b/src/tools/crushtool.cc
@@ -186,7 +186,7 @@ int main(int argc, const char **argv)
 
   CrushWrapper crush;
 
-  CrushTester tester(crush, cerr, 1);
+  CrushTester tester(crush, cerr);
 
   vector<const char *> empty_args;  // we use -c, don't confuse the generic arg parsing
   global_init(NULL, empty_args, CEPH_ENTITY_TYPE_CLIENT, CODE_ENVIRONMENT_UTILITY,