common/Graylog: consolidate ctors

* initialize member variables in-class
* use delegate ctor to dedup the logic in two ctors.

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

diff --git a/src/common/Graylog.cc b/src/common/Graylog.cc
index 3e9da5fe85a256894fbb9b5f74f9b178520a6ffc..cbd63fab25fdc4aa33385efe3072556dbf334ee2 100644 (file)
--- a/src/common/Graylog.cc
+++ b/src/common/Graylog.cc
@@ -13,9 +13,6 @@ namespace ceph::logging {
 
 Graylog::Graylog(const SubsystemMap * const s, const std::string &logger)
     : m_subs(s),
-      m_log_dst_valid(false),
-      m_hostname(""),
-      m_fsid(""),
       m_logger(std::move(logger)),
       m_ostream_compressed(std::stringstream::in |
                            std::stringstream::out |
@@ -26,18 +23,8 @@ Graylog::Graylog(const SubsystemMap * const s, const std::string &logger)
 }
 
 Graylog::Graylog(const std::string &logger)
-    : m_subs(NULL),
-      m_log_dst_valid(false),
-      m_hostname(""),
-      m_fsid(""),
-      m_logger(std::move(logger)),
-      m_ostream_compressed(std::stringstream::in |
-                           std::stringstream::out |
-                           std::stringstream::binary)
-{
-  m_formatter = std::unique_ptr<Formatter>(Formatter::create("json"));
-  m_formatter_section = std::unique_ptr<Formatter>(Formatter::create("json"));
-}
+  : Graylog(nullptr, logger)
+{}
 
 Graylog::~Graylog()
 {

diff --git a/src/common/Graylog.h b/src/common/Graylog.h
index a4214fa878927e648294b3c8ac1931ad15d472a4..c8c50131999d7b5d4b427d40e178ee0ad06b8b1c 100644 (file)
--- a/src/common/Graylog.h
+++ b/src/common/Graylog.h
@@ -59,7 +59,7 @@ class Graylog
  private:
   SubsystemMap const * const m_subs;
 
-  bool m_log_dst_valid;
+  bool m_log_dst_valid = false;
 
   std::string m_hostname;
   std::string m_fsid;