From 715d7d2aa32047f7c843c3495578435d88c4776f Mon Sep 17 00:00:00 2001
From: Greg Sjaardema <gsjaardema@gmail.com>
Date: Mon, 3 Jan 2022 14:16:31 -0700
Subject: [PATCH] Allow chrono to work with defined thousands separator

This allows the 8.1.0 and later chrono.h implementation to work (i.e. do not emit any undefined symbol warnings) if the user has `FMT_STATIC_THOUSANDS_SEPARATOR` defined.  The chrono implementation is relying on this definition of `locale_ref` to be defined.

Moving the `#ifndef` line to still define `locale_ref` and only select which version of `thousands_sep_impl` gets defined.
---
 include/fmt/format-inl.h | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/include/fmt/format-inl.h b/include/fmt/format-inl.h
index 04ded482..f2334e05 100644
--- a/include/fmt/format-inl.h
+++ b/include/fmt/format-inl.h
@@ -17,10 +17,7 @@
 #include <cstring>  // std::memmove
 #include <cwchar>
 #include <exception>
-
-#ifndef FMT_STATIC_THOUSANDS_SEPARATOR
-#  include <locale>
-#endif
+#include <locale>
 
 #ifdef _WIN32
 #  include <io.h>  // _isatty
@@ -96,7 +93,6 @@ inline void fwrite_fully(const void* ptr, size_t size, size_t count,
   if (written < count) FMT_THROW(system_error(errno, "cannot write to file"));
 }
 
-#ifndef FMT_STATIC_THOUSANDS_SEPARATOR
 template <typename Locale>
 locale_ref::locale_ref(const Locale& loc) : locale_(&loc) {
   static_assert(std::is_same<Locale, std::locale>::value, "");
@@ -107,6 +103,7 @@ template <typename Locale> Locale locale_ref::get() const {
   return locale_ ? *static_cast<const std::locale*>(locale_) : std::locale();
 }
 
+#ifndef FMT_STATIC_THOUSANDS_SEPARATOR
 template <typename Char>
 FMT_FUNC auto thousands_sep_impl(locale_ref loc) -> thousands_sep_result<Char> {
   auto& facet = std::use_facet<std::numpunct<Char>>(loc.get<std::locale>());