From: Kefu Chai <tchaikov@gmail.com>
Date: Tue, 26 Jul 2022 16:38:31 +0000 (+0800)
Subject: osd: use fmt/ranges.h for printing ranges
X-Git-Tag: v18.0.0~403^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F47286%2Fhead;p=ceph.git

osd: use fmt/ranges.h for printing ranges

now that fmt/ranges.h provides a generic mechinary for printing
ranges. let's just use it.

the output format is changed a little bit when it comes to
set and map, but as long as it makes sense, and we don't use
a tool to parse the logging messages for extracting information
from the messages printed by fmt::format(), we should be safe.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
---

diff --git a/src/common/hobject_fmt.h b/src/common/hobject_fmt.h
index e60fc2a14fc0..9c33f43446a8 100644
--- a/src/common/hobject_fmt.h
+++ b/src/common/hobject_fmt.h
@@ -9,7 +9,6 @@
 
 #include "common/hobject.h"
 #include "include/object_fmt.h"
-#include "include/types_fmt.h"
 #include "msg/msg_fmt.h"
 
 // \todo reimplement
diff --git a/src/include/types_fmt.h b/src/include/types_fmt.h
deleted file mode 100644
index a3ea24d99e70..000000000000
--- a/src/include/types_fmt.h
+++ /dev/null
@@ -1,67 +0,0 @@
-// -*- mode:C++; tab-width:8; c-basic-offset:2; indent-tabs-mode:t -*-
-// vim: ts=8 sw=2 smarttab
-#pragma once
-/**
- * \file fmtlib formatters for some types.h classes
- */
-
-#include <fmt/format.h>
-
-#include <string_view>
-
-#include "include/types.h"
-
-template <class Key, class T>
-struct fmt::formatter<std::pair<const Key, T>> {
-  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
-
-  template <typename FormatContext>
-  auto format(const std::pair<const Key, T>& p, FormatContext& ctx)
-  {
-    return fmt::format_to(ctx.out(), "{}={}", p.first, p.second);
-  }
-};
-
-template <class A, class B, class Comp, class Alloc>
-struct fmt::formatter<std::map<A, B, Comp, Alloc>> {
-  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
-
-  template <typename FormatContext>
-  auto format(const std::map<A, B, Comp, Alloc>& m, FormatContext& ctx)
-  {
-    return fmt::format_to(ctx.out(), "{{{}}}", fmt::join(m, ","));
-  }
-};
-
-template <class... Args>
-struct fmt::formatter<std::list<Args...>> {
-  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
-
-  template <typename FormatContext>
-  auto format(const std::list<Args...>& l, FormatContext& ctx)
-  {
-    return fmt::format_to(ctx.out(), "{}", fmt::join(l, ","));
-  }
-};
-
-template <class... Args>
-struct fmt::formatter<std::vector<Args...>> {
-  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
-
-  template <typename FormatContext>
-  auto format(const std::vector<Args...>& v, FormatContext& ctx)
-  {
-    return fmt::format_to(ctx.out(), "[{}]", fmt::join(v, ","));
-  }
-};
-
-template <class... Args>
-struct fmt::formatter<std::set<Args...>> {
-  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
-
-  template <typename FormatContext>
-  auto format(const std::set<Args...>& s, FormatContext& ctx)
-  {
-    return fmt::format_to(ctx.out(), "{}", fmt::join(s, ","));
-  }
-};
diff --git a/src/msg/msg_fmt.h b/src/msg/msg_fmt.h
index 5ed4ca997fe5..41c4c6af89de 100644
--- a/src/msg/msg_fmt.h
+++ b/src/msg/msg_fmt.h
@@ -8,7 +8,6 @@
 
 #include <fmt/format.h>
 
-#include "include/types_fmt.h"
 #include "msg/msg_types.h"
 
 template <>
diff --git a/src/osd/osd_types_fmt.h b/src/osd/osd_types_fmt.h
index cb54572c8e24..9124ac202085 100644
--- a/src/osd/osd_types_fmt.h
+++ b/src/osd/osd_types_fmt.h
@@ -6,7 +6,6 @@
  */
 
 #include "common/hobject_fmt.h"
-#include "include/types_fmt.h"
 #include "osd/osd_types.h"
 #include <fmt/chrono.h>
 
@@ -195,23 +194,6 @@ struct fmt::formatter<pg_info_t> {
 
 // snaps and snap-sets
 
-template <typename T, template <typename, typename, typename...> class C>
-struct fmt::formatter<interval_set<T, C>> {
-  constexpr auto parse(format_parse_context& ctx) { return ctx.begin(); }
-
-  template <typename FormatContext>
-  auto format(const interval_set<T, C>& inter, FormatContext& ctx)
-  {
-    bool first = true;
-    fmt::format_to(ctx.out(), "[");
-    for (const auto& [start, len] : inter) {
-      fmt::format_to(ctx.out(), "{}{}~{}", (first ? "" : ","), start, len);
-      first = false;
-    }
-    return fmt::format_to(ctx.out(), "]");
-  }
-};
-
 template <>
 struct fmt::formatter<SnapSet> {
   template <typename ParseContext>
diff --git a/src/osd/scrubber/pg_scrubber.cc b/src/osd/scrubber/pg_scrubber.cc
index f27a9c79a9c4..557e32e328ae 100644
--- a/src/osd/scrubber/pg_scrubber.cc
+++ b/src/osd/scrubber/pg_scrubber.cc
@@ -7,6 +7,8 @@
 #include <iostream>
 #include <vector>
 
+#include <fmt/ranges.h>
+
 #include "debug.h"
 
 #include "common/errno.h"
diff --git a/src/osd/scrubber/scrub_backend.cc b/src/osd/scrubber/scrub_backend.cc
index 70b3eed9ed0a..478eca12380f 100644
--- a/src/osd/scrubber/scrub_backend.cc
+++ b/src/osd/scrubber/scrub_backend.cc
@@ -5,6 +5,8 @@
 
 #include <algorithm>
 
+#include <fmt/ranges.h>
+
 #include "common/debug.h"
 
 #include "include/utime_fmt.h"
diff --git a/src/test/osd/scrubber_generators.cc b/src/test/osd/scrubber_generators.cc
index 4e84406c6789..0f2f371e714b 100644
--- a/src/test/osd/scrubber_generators.cc
+++ b/src/test/osd/scrubber_generators.cc
@@ -3,6 +3,8 @@
 
 #include "test/osd/scrubber_generators.h"
 
+#include <fmt/ranges.h>
+
 using namespace ScrubGenerator;
 
 // ref: PGLogTestRebuildMissing()
diff --git a/src/test/osd/test_scrubber_be.cc b/src/test/osd/test_scrubber_be.cc
index d42634020af9..1c7f0abab655 100644
--- a/src/test/osd/test_scrubber_be.cc
+++ b/src/test/osd/test_scrubber_be.cc
@@ -7,6 +7,8 @@
 #include <signal.h>
 #include <stdio.h>
 
+#include <fmt/ranges.h>
+
 #include "common/async/context_pool.h"
 #include "common/ceph_argparse.h"
 #include "global/global_context.h"