]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
libcephfs: build without "using namespace std"
authorKefu Chai <kchai@redhat.com>
Wed, 11 Aug 2021 04:00:00 +0000 (12:00 +0800)
committerKefu Chai <kchai@redhat.com>
Fri, 13 Aug 2021 04:23:37 +0000 (12:23 +0800)
* 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>
src/include/cephfs/metrics/Types.h
src/libcephfs.cc

index a004823c6a00a540d4dc2f2f7c6b3b6ee65d4551..2d944bb5b271722ed80782c089b8568745d4a36e 100644 (file)
@@ -67,7 +67,7 @@ struct ClientMetricPayloadBase {
     return metric_type;
   }
 
-  void print_type(ostream *out) const {
+  void print_type(std::ostream *out) const {
     *out << metric_type;
   }
 
@@ -111,7 +111,7 @@ struct CapInfoPayload : public ClientMetricPayloadBase {
     f->dump_int("num_caps", nr_caps);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "cap_hits: " << cap_hits << " "
         << "cap_misses: " << cap_misses << " "
         << "num_caps: " << nr_caps;
@@ -145,7 +145,7 @@ struct ReadLatencyPayload : public ClientMetricPayloadBase {
     f->dump_int("latency", lat);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "latency: " << lat;
   }
 };
@@ -177,7 +177,7 @@ struct WriteLatencyPayload : public ClientMetricPayloadBase {
     f->dump_int("latency", lat);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "latency: " << lat;
   }
 };
@@ -209,7 +209,7 @@ struct MetadataLatencyPayload : public ClientMetricPayloadBase {
     f->dump_int("latency", lat);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "latency: " << lat;
   }
 };
@@ -249,7 +249,7 @@ struct DentryLeasePayload : public ClientMetricPayloadBase {
     f->dump_int("num_dentries", nr_dentries);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "dlease_hits: " << dlease_hits << " "
         << "dlease_misses: " << dlease_misses << " "
         << "num_dentries: " << nr_dentries;
@@ -287,7 +287,7 @@ struct OpenedFilesPayload : public ClientMetricPayloadBase {
     f->dump_int("total_inodes", total_inodes);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "opened_files: " << opened_files << " "
         << "total_inodes: " << total_inodes;
   }
@@ -324,7 +324,7 @@ struct PinnedIcapsPayload : public ClientMetricPayloadBase {
     f->dump_int("total_inodes", total_inodes);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "pinned_icaps: " << pinned_icaps << " "
         << "total_inodes: " << total_inodes;
   }
@@ -361,7 +361,7 @@ struct OpenedInodesPayload : public ClientMetricPayloadBase {
     f->dump_int("total_inodes", total_inodes);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     *out << "opened_inodes: " << opened_inodes << " "
         << "total_inodes: " << total_inodes;
   }
@@ -386,7 +386,7 @@ struct UnknownPayload : public ClientMetricPayloadBase {
   void dump(Formatter *f) const {
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
   }
 };
 
@@ -455,7 +455,7 @@ public:
 
   class PrintPayloadVisitor : public boost::static_visitor<void> {
   public:
-    explicit PrintPayloadVisitor(ostream *out) : _out(out) {
+    explicit PrintPayloadVisitor(std::ostream *out) : _out(out) {
     }
 
     template <typename ClientMetricPayload>
@@ -468,7 +468,7 @@ public:
     }
 
   private:
-    ostream *_out;
+    std::ostream *_out;
   };
 
   void encode(bufferlist &bl) const {
@@ -518,7 +518,7 @@ public:
     apply_visitor(DumpPayloadVisitor(f), payload);
   }
 
-  void print(ostream *out) const {
+  void print(std::ostream *out) const {
     apply_visitor(PrintPayloadVisitor(out), payload);
   }
 
index a379b7b6bf78043884a92a4702bfc7c60f1cb084..63af614bcefdbde3370cc68f23f18a51c93523d7 100644 (file)
@@ -37,6 +37,8 @@
 
 #define DEFAULT_UMASK 002
 
+using namespace std;
+
 static mode_t umask_cb(void *);
 namespace {
 // Set things up this way so we don't start up threads until mount and