common/fmt_common.h: move optional<T> formatter from pg.cc

Signed-off-by: Samuel Just <sjust@redhat.com>

diff --git a/src/common/fmt_common.h b/src/common/fmt_common.h
index d68d6457dcb886367fe72aa2e3b6bb94f521f751..27e488aedcebfd2ea1b1da6cc1f3cff10bae6799 100644 (file)
--- a/src/common/fmt_common.h
+++ b/src/common/fmt_common.h
@@ -2,6 +2,8 @@
 // vim: ts=8 sw=2 smarttab
 #pragma once
 
+#include <optional>
+
 /**
  * \file default fmtlib formatters for specifically-tagged types
  */
@@ -61,4 +63,17 @@ struct formatter<T> {
   }
   bool verbose{true};
 };
+
+template <typename T>
+struct formatter<std::optional<T>> {
+  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
+  template <typename FormatContext>
+  auto format(const std::optional<T> &v, FormatContext& ctx) const {
+    if (v.has_value()) {
+      return fmt::format_to(ctx.out(), "{}", *v);
+    }
+    return fmt::format_to(ctx.out(), "<null>");
+  }
+};
+
 }  // namespace fmt

diff --git a/src/crimson/osd/pg.cc b/src/crimson/osd/pg.cc
index e303ecf9a45c8e5de9f9ebbcd54ee7ae114eb936..013f564b5f12aaabe440abc863585bfea4ecd468 100644 (file)
--- a/src/crimson/osd/pg.cc
+++ b/src/crimson/osd/pg.cc
@@ -64,17 +64,6 @@ std::ostream& operator<<(std::ostream& out, const signedspan& d)
 }
 }
 
-template <typename T>
-struct fmt::formatter<std::optional<T>> : fmt::formatter<T> {
-  template <typename FormatContext>
-  auto format(const std::optional<T>& v, FormatContext& ctx) const {
-    if (v.has_value()) {
-      return fmt::formatter<T>::format(*v, ctx);
-    }
-    return fmt::format_to(ctx.out(), "<null>");
-  }
-};
-
 namespace crimson::osd {
 
 using crimson::common::local_conf;