src: put-to operator function - const input cleanup

Fixed the instances of the issue specified in Tracker#3977 inside src/ folder utmost, by adding
const input. This fix avoids the following directories inside src: rocksdb/ zstd/ boost/ rapidjson/ googletest/ Beast/.

Fixes: http://tracker.ceph.com/issues/3977
Signed-off-by: Jos Collin <jcollin@redhat.com>

diff --git a/src/common/LogEntry.h b/src/common/LogEntry.h
index afe8533acd432af222ac4fa01ee88292a9ebfee5..6f213dfce65a4cf5b0d9eed537676f4fd9babca0 100644 (file)
--- a/src/common/LogEntry.h
+++ b/src/common/LogEntry.h
@@ -119,7 +119,7 @@ struct LogSummary {
 };
 WRITE_CLASS_ENCODER_FEATURES(LogSummary)
 
-inline ostream& operator<<(ostream& out, clog_type t)
+inline ostream& operator<<(ostream& out, const clog_type t)
 {
   switch (t) {
   case CLOG_DEBUG:

diff --git a/src/common/code_environment.cc b/src/common/code_environment.cc
index d3dbda5730100406cc26f6786a9ffbafec3e2d7c..e75e561581d0164d21ca31d130390603dee6ba5a 100644 (file)
--- a/src/common/code_environment.cc
+++ b/src/common/code_environment.cc
@@ -41,7 +41,7 @@ extern "C" const char *code_environment_to_str(enum code_environment_t e)
   }
 }
 
-std::ostream &operator<<(std::ostream &oss, enum code_environment_t e)
+std::ostream &operator<<(std::ostream &oss, const enum code_environment_t e)
 {
   oss << code_environment_to_str(e);
   return oss;

diff --git a/src/common/code_environment.h b/src/common/code_environment.h
index e21f0eff4f878cd4834e0dd6b4f2380d214d544c..b94ba52caefc366bacc0de7248334ec6da98bae2 100644 (file)
--- a/src/common/code_environment.h
+++ b/src/common/code_environment.h
@@ -28,14 +28,14 @@ enum code_environment_t {
 
 extern "C" code_environment_t g_code_env;
 extern "C" const char *code_environment_to_str(enum code_environment_t e);
-std::ostream &operator<<(std::ostream &oss, enum code_environment_t e);
+std::ostream &operator<<(std::ostream &oss, const enum code_environment_t e);
 extern "C" int get_process_name(char *buf, int len);
 std::string get_process_name_cpp();
 
 #else
 
 extern code_environment_t g_code_env;
-const char *code_environment_to_str(enum code_environment_t e);
+const char *code_environment_to_str(const enum code_environment_t e);
 extern int get_process_name(char *buf, int len);
 
 #endif

diff --git a/src/include/buffer.h b/src/include/buffer.h
index 177e95f8a8753e3441c1d0dae8a440ac4b345218..e86b69dcc497d3f9115209c93752ee34abd4d708 100644 (file)
--- a/src/include/buffer.h
+++ b/src/include/buffer.h
@@ -904,7 +904,7 @@ namespace buffer CEPH_BUFFER_API {
     // cppcheck-suppress noExplicitConstructor
     hash(uint32_t init) : crc(init) { }
 
-    void update(buffer::list& bl) {
+    void update(const buffer::list& bl) {
       crc = bl.crc32c(crc);
     }
 
@@ -956,7 +956,7 @@ std::ostream& operator<<(std::ostream& out, const buffer::list& bl);
 
 std::ostream& operator<<(std::ostream& out, const buffer::error& e);
 
-inline bufferhash& operator<<(bufferhash& l, bufferlist &r) {
+inline bufferhash& operator<<(bufferhash& l, const bufferlist &r) {
   l.update(r);
   return l;
 }

diff --git a/src/include/frag.h b/src/include/frag.h
index 7e0d4a8eefb7b33ec483346464f13dfadffc2b92..3b1456cbbe737ceab25ccc86d706934287a56ca7 100644 (file)
--- a/src/include/frag.h
+++ b/src/include/frag.h
@@ -148,7 +148,7 @@ class frag_t {
   }
 };
 
-inline std::ostream& operator<<(std::ostream& out, frag_t hb)
+inline std::ostream& operator<<(std::ostream& out, const frag_t& hb)
 {
   //out << std::hex << hb.value() << std::dec << "/" << hb.bits() << '=';
   unsigned num = hb.bits();

diff --git a/src/include/fs_types.h b/src/include/fs_types.h
index dba085ad13e038d76cca63a99b7d10a9a2dd212a..c6343fe1bcd14765725d98ccd573d9c715d17a0d 100644 (file)
--- a/src/include/fs_types.h
+++ b/src/include/fs_types.h
@@ -44,7 +44,7 @@ struct denc_traits<inodeno_t> {
   }
 };
 
-inline ostream& operator<<(ostream& out, inodeno_t ino) {
+inline ostream& operator<<(ostream& out, const inodeno_t& ino) {
   return out << hex << ino.val << dec;
 }
 

diff --git a/src/include/object.h b/src/include/object.h
index 1bad9ea24bcfb89de06fda7be9150cb99d16f447..50aa3fee240ec6a90d66e2a6923bcc95bd8a42f4 100644 (file)
--- a/src/include/object.h
+++ b/src/include/object.h
@@ -139,7 +139,7 @@ struct denc_traits<snapid_t> {
   }
 };
 
-inline ostream& operator<<(ostream& out, snapid_t s) {
+inline ostream& operator<<(ostream& out, const snapid_t& s) {
   if (s == CEPH_NOSNAP)
     return out << "head";
   else if (s == CEPH_SNAPDIR)

diff --git a/src/include/types.h b/src/include/types.h
index 1f0ee2b03a35a86599eb2b30956e61145a7acbee..f859828e8c0e49203b328cdc59a2a9e8a5542b87 100644 (file)
--- a/src/include/types.h
+++ b/src/include/types.h
@@ -418,7 +418,7 @@ enum health_status_t {
 };
 
 #ifdef __cplusplus
-inline ostream& operator<<(ostream &oss, health_status_t status) {
+inline ostream& operator<<(ostream &oss, const health_status_t status) {
   switch (status) {
     case HEALTH_ERR:
       oss << "HEALTH_ERR";

diff --git a/src/librbd/internal.cc b/src/librbd/internal.cc
index b790b3539f76a372ad1faf65dff6aa80c6de7f4b..00606fcd2e6d81c5b12bc3a021629e81e0586fdc 100644 (file)
--- a/src/librbd/internal.cc
+++ b/src/librbd/internal.cc
@@ -327,7 +327,7 @@ int validate_pool(IoCtx &io_ctx, CephContext *cct) {
     }
   }
 
-  std::ostream &operator<<(std::ostream &os, rbd_image_options_t &opts) {
+  std::ostream &operator<<(std::ostream &os, const rbd_image_options_t &opts) {
     image_options_ref* opts_ = static_cast<image_options_ref*>(opts);
 
     os << "[";
@@ -343,7 +343,7 @@ int validate_pool(IoCtx &io_ctx, CephContext *cct) {
     return os;
   }
 
-  std::ostream &operator<<(std::ostream &os, ImageOptions &opts) {
+  std::ostream &operator<<(std::ostream &os, const ImageOptions &opts) {
     os << "[";
 
     const char *delimiter = "";

diff --git a/src/mds/MDSCacheObject.h b/src/mds/MDSCacheObject.h
index ec40359dd762b15c80fbe083c34d313f01be0769..1bc80cfed67307f6f22792f44e1d397e35022907 100644 (file)
--- a/src/mds/MDSCacheObject.h
+++ b/src/mds/MDSCacheObject.h
@@ -48,10 +48,10 @@ struct mdsco_db_line_prefix {
   MDSCacheObject *object;
   explicit mdsco_db_line_prefix(MDSCacheObject *o) : object(o) {}
 };
-std::ostream& operator<<(std::ostream& out, mdsco_db_line_prefix o);
+std::ostream& operator<<(std::ostream& out, const mdsco_db_line_prefix& o);
 
 // printer
-std::ostream& operator<<(std::ostream& out, MDSCacheObject &o);
+std::ostream& operator<<(std::ostream& out, const MDSCacheObject &o);
 
 class MDSCacheObject {
  public:
@@ -406,7 +406,7 @@ inline std::ostream& operator<<(std::ostream& out, MDSCacheObject &o) {
   return out;
 }
 
-inline std::ostream& operator<<(std::ostream& out, mdsco_db_line_prefix o) {
+inline std::ostream& operator<<(std::ostream& out, const mdsco_db_line_prefix& o) {
   o.object->print_db_line_prefix(out);
   return out;
 }

diff --git a/src/messages/MMDSResolve.h b/src/messages/MMDSResolve.h
index 6e9f028529752b27492ac87a16ea254cd788977c..baf581b4bd0bca930e4ecf10e45c8ad1115b47a7 100644 (file)
--- a/src/messages/MMDSResolve.h
+++ b/src/messages/MMDSResolve.h
@@ -86,7 +86,7 @@ public:
   }
 };
 
-inline ostream& operator<<(ostream& out, const MMDSResolve::slave_request) {
+inline ostream& operator<<(ostream& out, const MMDSResolve::slave_request&) {
     return out;
 }
 

diff --git a/src/mon/MonCap.cc b/src/mon/MonCap.cc
index ec335752cdd2c333c212c970872f72b94ada2d66..2ba68b71f796ebf5b8c34b9cc9ed3aa05c900534 100644 (file)
--- a/src/mon/MonCap.cc
+++ b/src/mon/MonCap.cc
@@ -44,7 +44,7 @@ using std::vector;
 
 #define dout_subsys ceph_subsys_mon
 
-ostream& operator<<(ostream& out, mon_rwxa_t p)
+ostream& operator<<(ostream& out, const mon_rwxa_t& p)
 { 
   if (p == MON_CAP_ANY)
     return out << "*";

diff --git a/src/mon/MonCap.h b/src/mon/MonCap.h
index 25684d307624d8bf91edee790e7f5d8326020d15..46acb1e42b5fae68319ae1b10479a3ac42cb0398 100644 (file)
--- a/src/mon/MonCap.h
+++ b/src/mon/MonCap.h
@@ -32,7 +32,7 @@ struct mon_rwxa_t {
   }
 };
 
-ostream& operator<<(ostream& out, mon_rwxa_t p);
+ostream& operator<<(ostream& out, const mon_rwxa_t& p);
 
 struct StringConstraint {
   string value;

diff --git a/src/msg/async/dpdk/IP.cc b/src/msg/async/dpdk/IP.cc
index 3d3e8fb50bca3838da0be4f0fde7dc95a539a5c4..9f36d2eafffdd208d2eb7caeb87218b1ccb22257 100644 (file)
--- a/src/msg/async/dpdk/IP.cc
+++ b/src/msg/async/dpdk/IP.cc
@@ -48,7 +48,7 @@
 #undef dout_prefix
 #define dout_prefix *_dout << "dpdk "
 
-std::ostream& operator<<(std::ostream& os, ipv4_address a) {
+std::ostream& operator<<(std::ostream& os, const ipv4_address& a) {
   auto ip = a.ip;
   return os << ((ip >> 24) & 0xff) << "." << ((ip >> 16) & 0xff)
             << "." << ((ip >> 8) & 0xff) << "." << ((ip >> 0) & 0xff);

diff --git a/src/msg/async/dpdk/TCP.h b/src/msg/async/dpdk/TCP.h
index eb3c10490e645ad8e5f85b10a78a2def40d420ef..9923fab544db9636a915f6335c887d611922075e 100644 (file)
--- a/src/msg/async/dpdk/TCP.h
+++ b/src/msg/async/dpdk/TCP.h
@@ -69,7 +69,7 @@ inline tcp_state operator|(tcp_state s1, tcp_state s2) {
   return tcp_state(uint16_t(s1) | uint16_t(s2));
 }
 
-inline std::ostream & operator<<(std::ostream & str, tcp_state s) {
+inline std::ostream & operator<<(std::ostream & str, const tcp_state& s) {
   switch (s) {
     case tcp_state::CLOSED: return str << "CLOSED";
     case tcp_state::LISTEN: return str << "LISTEN";
@@ -154,7 +154,7 @@ tcp_sequence hton(tcp_sequence ts) {
   return tcp_sequence { ::hton(ts.raw) };
 }
 
-inline std::ostream& operator<<(std::ostream& os, tcp_sequence s) {
+inline std::ostream& operator<<(std::ostream& os, const tcp_sequence& s) {
   return os << s.raw;
 }
 

diff --git a/src/msg/async/dpdk/ethernet.cc b/src/msg/async/dpdk/ethernet.cc
index e1bf0466581bf0732f4d13474011383e7c7a730a..9aca5078871688dae4db023d13aa0df3b0dd3ee8 100644 (file)
--- a/src/msg/async/dpdk/ethernet.cc
+++ b/src/msg/async/dpdk/ethernet.cc
@@ -2,7 +2,7 @@
 
 #include "ethernet.h"
 
-std::ostream& operator<<(std::ostream& os, ethernet_address ea) {
+std::ostream& operator<<(std::ostream& os, const ethernet_address& ea) {
   auto& m = ea.mac;
   using u = uint32_t;
   os << std::hex << std::setw(2)

diff --git a/src/msg/async/dpdk/ethernet.h b/src/msg/async/dpdk/ethernet.h
index b602de62c123e84bdc59c3e2be7cfea38c738697..17546d73a10e7c69642b813b34f4818a0b835bc1 100644 (file)
--- a/src/msg/async/dpdk/ethernet.h
+++ b/src/msg/async/dpdk/ethernet.h
@@ -53,7 +53,7 @@ struct ethernet_address {
 inline bool operator==(const ethernet_address& a, const ethernet_address& b) {
   return a.mac == b.mac;
 }
-std::ostream& operator<<(std::ostream& os, ethernet_address ea);
+std::ostream& operator<<(std::ostream& os, const ethernet_address& ea);
 
 struct ethernet {
   using address = ethernet_address;

diff --git a/src/msg/async/dpdk/ip_types.h b/src/msg/async/dpdk/ip_types.h
index 55e0bcfb22687290a86c811882d78f7196caf18c..356d8fd6eac8e9486c07c72fec8c6c342da5d080 100644 (file)
--- a/src/msg/async/dpdk/ip_types.h
+++ b/src/msg/async/dpdk/ip_types.h
@@ -95,7 +95,7 @@ struct ipv4_address {
 
 static inline bool is_unspecified(ipv4_address addr) { return addr.ip == 0; }
 
-std::ostream& operator<<(std::ostream& os, ipv4_address a);
+std::ostream& operator<<(std::ostream& os, const ipv4_address& a);
 
 namespace std {
 

diff --git a/src/os/bluestore/bluefs_types.cc b/src/os/bluestore/bluefs_types.cc
index e414c8b6d5be7942d24dcd55a9a5e9062e960abe..93d79ffec4841dc17005e899c3218028aaa9eb70 100644 (file)
--- a/src/os/bluestore/bluefs_types.cc
+++ b/src/os/bluestore/bluefs_types.cc
@@ -23,7 +23,7 @@ void bluefs_extent_t::generate_test_instances(list<bluefs_extent_t*>& ls)
   ls.back()->bdev = 1;
 }
 
-ostream& operator<<(ostream& out, bluefs_extent_t e)
+ostream& operator<<(ostream& out, const bluefs_extent_t& e)
 {
   return out << (int)e.bdev << ":0x" << std::hex << e.offset << "+" << e.length
             << std::dec;

diff --git a/src/os/bluestore/bluefs_types.h b/src/os/bluestore/bluefs_types.h
index 6a015892d70b14482cf5cd690913aa4fa3897541..ddd55def0549fd9c87328762af8c97b1f9820e50 100644 (file)
--- a/src/os/bluestore/bluefs_types.h
+++ b/src/os/bluestore/bluefs_types.h
@@ -28,7 +28,7 @@ public:
 };
 WRITE_CLASS_DENC(bluefs_extent_t)
 
-ostream& operator<<(ostream& out, bluefs_extent_t e);
+ostream& operator<<(ostream& out, const bluefs_extent_t& e);
 
 
 struct bluefs_fnode_t {

diff --git a/src/osd/OSDCap.cc b/src/osd/OSDCap.cc
index c8cfbfab1f366a45a0dba0204d78bef773360937..25da5e2b69a5dd76f6115a638fffa5bc8686e291 100644 (file)
--- a/src/osd/OSDCap.cc
+++ b/src/osd/OSDCap.cc
@@ -24,7 +24,7 @@
 using std::ostream;
 using std::vector;
 
-ostream& operator<<(ostream& out, osd_rwxa_t p)
+ostream& operator<<(ostream& out, const osd_rwxa_t& p)
 { 
   if (p == OSD_CAP_ANY)
     return out << "*";

diff --git a/src/osd/OSDCap.h b/src/osd/OSDCap.h
index 3a1a722aaa7144b4c7567cdada9ec0b2454971dd..8ce808052eb5f64c75ed1275496d187846cf6a7f 100644 (file)
--- a/src/osd/OSDCap.h
+++ b/src/osd/OSDCap.h
@@ -54,7 +54,7 @@ struct osd_rwxa_t {
   }
 };
 
-ostream& operator<<(ostream& out, osd_rwxa_t p);
+ostream& operator<<(ostream& out, const osd_rwxa_t& p);
 
 struct OSDCapSpec {
   osd_rwxa_t allow;