From: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
Date: Wed, 27 Feb 2013 17:27:56 +0000 (+0100)
Subject: CrushCompiler.cc: remove duplicate if/else branches
X-Git-Tag: v0.59~84^2~5
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=afa811caa8178cf22c0a119a596f9d2f40225dc9;p=ceph.git

CrushCompiler.cc: remove duplicate if/else branches

Fix duplicate content of a if/else branch. Remove the
complete if/else around the content.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
---

diff --git a/src/crush/CrushCompiler.cc b/src/crush/CrushCompiler.cc
index 5a9eb441b2c..c54a7b916e4 100644
--- a/src/crush/CrushCompiler.cc
+++ b/src/crush/CrushCompiler.cc
@@ -100,12 +100,9 @@ int CrushCompiler::decompile_bucket_impl(int i, ostream &out)
     print_item_name(out, item, crush);
     out << " weight ";
     print_fixedpoint(out, w);
-    if (dopos) {
-      if (alg == CRUSH_BUCKET_TREE)
-	out << " pos " << j;
-      else
-	out << " pos " << j;
-    }
+    if (dopos) 
+      out << " pos " << j;
+    
     out << "\n";
   }
   out << "}\n";