rbd_replay: build without "using namespace std"

* add "std::" prefix in headers
* add "using" declarations in .cc files.

so we don't rely on "using namespace std" in one or more included
headers.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/rbd_replay/Replayer.cc b/src/rbd_replay/Replayer.cc
index 11e84900d1a3984c5dde63fe3bb94981a8ed2dd9..8e7bd0c1917e9a900c1e06e7fbfafea0e8173047 100644 (file)
--- a/src/rbd_replay/Replayer.cc
+++ b/src/rbd_replay/Replayer.cc
@@ -27,6 +27,8 @@
 
 #define dout_context g_ceph_context
 
+using std::cerr;
+using std::cout;
 using namespace rbd_replay;
 
 namespace {
@@ -99,7 +101,7 @@ void Worker::run() {
     while (!m_pending_ios.empty()) {
       if (!first_time) {
        dout(THREAD_LEVEL) << "Worker thread trying to stop, still waiting for " << m_pending_ios.size() << " pending IOs to complete:" << dendl;
-       pair<action_id_t, PendingIO::ptr> p;
+       std::pair<action_id_t, PendingIO::ptr> p;
        BOOST_FOREACH(p, m_pending_ios) {
          dout(THREAD_LEVEL) << "> " << p.first << dendl;
        }
@@ -157,7 +159,7 @@ bool Worker::readonly() const {
   return m_replayer.readonly();
 }
 
-rbd_loc Worker::map_image_name(string image_name, string snap_name) const {
+rbd_loc Worker::map_image_name(std::string image_name, std::string snap_name) const {
   return m_replayer.image_name_map().map(rbd_loc("", image_name, snap_name));
 }
 
@@ -212,7 +214,7 @@ void Replayer::run(const std::string& replay_file) {
        goto out2;
       }
       m_rbd = new librbd::RBD();
-      map<thread_id_t, Worker*> workers;
+      std::map<thread_id_t, Worker*> workers;
 
       int fd = open(replay_file.c_str(), O_RDONLY|O_BINARY);
       if (fd < 0) {
@@ -264,7 +266,7 @@ void Replayer::run(const std::string& replay_file) {
       }
 
       dout(THREAD_LEVEL) << "Waiting for workers to die" << dendl;
-      pair<thread_id_t, Worker*> w;
+      std::pair<thread_id_t, Worker*> w;
       BOOST_FOREACH(w, workers) {
        w.second->join();
        delete w.second;
@@ -298,11 +300,11 @@ void Replayer::erase_image(imagectx_id_t imagectx_id) {
   std::unique_lock lock{m_images_mutex};
   librbd::Image* image = m_images[imagectx_id];
   if (m_dump_perf_counters) {
-    string command = "perf dump";
+    std::string command = "perf dump";
     cmdmap_t cmdmap;
     JSONFormatter jf(true);
     bufferlist out;
-    stringstream ss;
+    std::stringstream ss;
     g_ceph_context->do_command(command, cmdmap, &jf, ss, &out);
     jf.flush(cout);
     cout << std::endl;
@@ -368,11 +370,11 @@ void Replayer::wait_for_actions(const action::Dependencies &deps) {
 void Replayer::clear_images() {
   std::shared_lock lock{m_images_mutex};
   if (m_dump_perf_counters && !m_images.empty()) {
-    string command = "perf dump";
+    std::string command = "perf dump";
     cmdmap_t cmdmap;
     JSONFormatter jf(true);
     bufferlist out;
-    stringstream ss;
+    std::stringstream ss;
     g_ceph_context->do_command(command, cmdmap, &jf, ss, &out);
     jf.flush(cout);
     cout << std::endl;
@@ -397,10 +399,10 @@ void Replayer::set_readonly(bool readonly) {
   m_readonly = readonly;
 }
 
-string Replayer::pool_name() const {
+std::string Replayer::pool_name() const {
   return m_pool_name;
 }
 
-void Replayer::set_pool_name(string pool_name) {
+void Replayer::set_pool_name(std::string pool_name) {
   m_pool_name = pool_name;
 }

diff --git a/src/rbd_replay/actions.cc b/src/rbd_replay/actions.cc
index 3a95c399f08290293a108887a39c69063bfcaabe..8aae77c09651b18bd742480633e1b07e4eb137c3 100644 (file)
--- a/src/rbd_replay/actions.cc
+++ b/src/rbd_replay/actions.cc
@@ -20,6 +20,7 @@
 
 #define dout_context g_ceph_context
 
+using std::cerr;
 using namespace rbd_replay;
 
 namespace {

diff --git a/src/rbd_replay/rbd-replay.cc b/src/rbd_replay/rbd-replay.cc
index f52e75fd98a3d51b974ca59f020d9668e94348a6..2f13cf3121186b39e44a4490fd85c00613cb1fd0 100644 (file)
--- a/src/rbd_replay/rbd-replay.cc
+++ b/src/rbd_replay/rbd-replay.cc
@@ -20,7 +20,7 @@
 #include "rbd_replay_debug.hpp"
 #include "ImageNameMap.hpp"
 
-
+using namespace std;
 using namespace rbd_replay;