common/Formatter: make get_attrs_str() const

This getter function never modifies `this`.

Signed-off-by: Max Kellermann <[email protected]>

Author: MaxKellermann

src/common/Formatter.cc

@@ -582,7 +582,7 @@ void XMLFormatter::write_bin_data(const char* buff, int buf_len)
   m_ss.seekg(buf_len);
 }
 
-void XMLFormatter::get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str)
+void XMLFormatter::get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str) const
 {
   CachedStackStringStream css;
 
@@ -959,7 +959,7 @@ void TableFormatter::write_raw_data(const char *data) {
   // not supported
 }
 
-void TableFormatter::get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str)
+void TableFormatter::get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str) const
 {
   CachedStackStringStream css;
 

src/common/Formatter.h

@@ -328,7 +328,7 @@ namespace ceph {
     void open_section_in_ns(std::string_view name, const char *ns, const FormatterAttrs *attrs);
     void finish_pending_string();
     void print_spaces();
-    void get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str);
+    void get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str) const;
     char to_lower_underscore(char c) const;
     std::string get_xml_name(std::string_view name) const;
 
@@ -376,7 +376,7 @@ namespace ceph {
 
     int get_len() const override;
     void write_raw_data(const char *data) override;
-    void get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str);
+    void get_attrs_str(const FormatterAttrs *attrs, std::string& attrs_str) const;
 
   private:
     template <class T>