client: make _do_lookup take a const string

Signed-off-by: Sage Weil <sage@inktank.com>

diff --git a/src/client/Client.cc b/src/client/Client.cc
index c57b7131fdcd46c189dbc90e6018bec1ee2a3ea4..9f8ba4accbcd13056d95eee1c51d64321e04f05a 100644 (file)
--- a/src/client/Client.cc
+++ b/src/client/Client.cc
@@ -3829,7 +3829,7 @@ void Client::renew_caps(const int mds) {
 // ===============================================================
 // high level (POSIXy) interface
 
-int Client::_do_lookup(Inode *dir, const char *name, Inode **target)
+int Client::_do_lookup(Inode *dir, const string& name, Inode **target)
 {
   int op = dir->snapid == CEPH_SNAPDIR ? CEPH_MDS_OP_LOOKUPSNAP : CEPH_MDS_OP_LOOKUP;
   MetaRequest *req = new MetaRequest(op);
@@ -3839,10 +3839,10 @@ int Client::_do_lookup(Inode *dir, const char *name, Inode **target)
   req->set_filepath(path);
   req->inode = dir;
   req->head.args.getattr.mask = 0;
-  ldout(cct, 10) << "_lookup on " << path << dendl;
+  ldout(cct, 10) << "_do_lookup on " << path << dendl;
 
   int r = make_request(req, 0, 0, target);
-  ldout(cct, 10) << "_lookup res is " << r << dendl;
+  ldout(cct, 10) << "_do_lookup res is " << r << dendl;
   return r;
 }
 
@@ -3921,7 +3921,7 @@ int Client::_lookup(Inode *dir, const string& dname, Inode **target)
     }
   }
 
-  r = _do_lookup(dir, dname.c_str(), target);
+  r = _do_lookup(dir, dname, target);
 
  done:
   if (r < 0)

diff --git a/src/client/Client.h b/src/client/Client.h
index ba6bb7c4a5fa95c18ce967ad12f6eec5581bb469..7c404c214da55ac1c960c9752c578379960ba1b9 100644 (file)
--- a/src/client/Client.h
+++ b/src/client/Client.h
@@ -521,7 +521,7 @@ private:
 
   // internal interface
   //   call these with client_lock held!
-  int _do_lookup(Inode *dir, const char *name, Inode **target);
+  int _do_lookup(Inode *dir, const string& name, Inode **target);
   int _lookup(Inode *dir, const string& dname, Inode **target);
 
   int _link(Inode *in, Inode *dir, const char *name, int uid=-1, int gid=-1, Inode **inp = 0);