From 2c3a5698ef4956568d378148b6e271dd70868669 Mon Sep 17 00:00:00 2001
From: Vladislav Shchapov <vladislav@shchapov.ru>
Date: Sat, 4 Jan 2025 19:02:51 +0500
Subject: [PATCH] Simplify a copying the fill from basic_specs

Signed-off-by: Vladislav Shchapov <vladislav@shchapov.ru>
---
 include/fmt/base.h   |  6 ++++++
 include/fmt/format.h | 13 +------------
 include/fmt/std.h    |  2 +-
 3 files changed, 8 insertions(+), 13 deletions(-)

diff --git a/include/fmt/base.h b/include/fmt/base.h
index 00c51423eeca..a6948d40b462 100644
--- a/include/fmt/base.h
+++ b/include/fmt/base.h
@@ -842,6 +842,12 @@ class basic_specs {
     for (size_t i = 0; i < size; ++i)
       fill_data_[i & 3] = static_cast<char>(s[i]);
   }
+
+  FMT_CONSTEXPR void set_fill(const basic_specs& specs) {
+    set_fill_size(specs.fill_size());
+    for (size_t i = 0; i < max_fill_size; ++i)
+      fill_data_[i] = specs.fill_data_[i];
+  }
 };
 
 // Format specifiers for built-in and string types.
diff --git a/include/fmt/format.h b/include/fmt/format.h
index 2fb85fef0350..c9a6054d1957 100644
--- a/include/fmt/format.h
+++ b/include/fmt/format.h
@@ -3654,17 +3654,6 @@ void vformat_to(buffer<Char>& buf, basic_string_view<Char> fmt,
   parse_format_string(
       fmt, format_handler<Char>{parse_context<Char>(fmt), {out, args, loc}});
 }
-
-template <typename Char>
-void basic_specs_copy_fill(basic_specs& dst, const basic_specs& src) {
-  if (src.fill_size() == 1 && const_check(!std::is_same<Char, char>::value)) {
-    Char fill = src.fill_unit<Char>();
-    dst.set_fill(basic_string_view<Char>(&fill, 1));
-    return;
-  }
-  dst.set_fill(basic_string_view<char>(src.fill<char>(), src.fill_size()));
-}
-
 }  // namespace detail
 
 FMT_BEGIN_EXPORT
@@ -3971,7 +3960,7 @@ template <typename T, typename Char = char> struct nested_formatter {
     write(basic_appender<Char>(buf));
     auto specs = format_specs();
     specs.width = width_;
-    detail::basic_specs_copy_fill<Char>(specs, specs_);
+    specs.set_fill(specs_);
     specs.set_align(specs_.align());
     return detail::write<Char>(
         ctx.out(), basic_string_view<Char>(buf.data(), buf.size()), specs);
diff --git a/include/fmt/std.h b/include/fmt/std.h
index bc277d646267..bb07fedc0c1b 100644
--- a/include/fmt/std.h
+++ b/include/fmt/std.h
@@ -696,7 +696,7 @@ template <typename T, typename Char> struct formatter<std::complex<T>, Char> {
 
     auto outer_specs = format_specs();
     outer_specs.width = specs.width;
-    detail::basic_specs_copy_fill<Char>(outer_specs, specs);
+    outer_specs.set_fill(specs);
     outer_specs.set_align(specs.align());
 
     specs.width = 0;