mds: remove useless code

As scrub_dentry_next() never returns a negative result code,
it always returns 0, EAGAIN, ENOENT.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>

diff --git a/src/mds/ScrubStack.cc b/src/mds/ScrubStack.cc
index 06fada821fcf2dd7b1fa8f96cd50851e8fd0c8f6..77525f3d3b3b2eaee487ba1319e2a60b2ba1aa11 100644 (file)
--- a/src/mds/ScrubStack.cc
+++ b/src/mds/ScrubStack.cc
@@ -345,18 +345,7 @@ void ScrubStack::scrub_dirfrag(CDir *dir,
       return;
     }
 
-    if (r < 0) {
-      // FIXME: how can I handle an error here?  I can't hold someone up
-      // forever, but I can't say "sure you're scrubbed"
-      //  -- should change scrub_dentry_next definition to never
-      //  give out IO errors (handle them some other way)
-      //     
-      derr << __func__ << " error from scrub_dentry_next: "
-           << r << dendl;
-      return;
-    }
-
-    // scrub_dentry_next defined to only give -ve, EAGAIN, ENOENT, 0 -- we should
+    // scrub_dentry_next defined to only give EAGAIN, ENOENT, 0 -- we should
     // never get random IO errors here.
     assert(r == 0);