From e6e1ca5a4e80893ab8363c54cfd2467b6ddd3e7d Mon Sep 17 00:00:00 2001
From: Odysseas Georgoudis <odygrd@gmail.com>
Date: Sat, 2 Nov 2024 03:10:23 +0000
Subject: [PATCH] rename fmt PACKED (#624)

---
 CHANGELOG.md                      | 3 +++
 include/quill/bundled/fmt/base.h  | 4 ++--
 include/quill/sinks/ConsoleSink.h | 4 ++--
 3 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index fe038170..9d645116 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -97,6 +97,9 @@
 - The `set_thread_name` function has been fixed to provide accurate error reporting, ensuring that the correct error
   message is displayed in the event of a failure.
 
+- Renamed `PACKED` used in `libfmt` to `QUILLPACKED` to avoid naming
+  collisions. ([#620](https://github.com/odygrd/quill/issues/620))
+
 ## v7.4.0
 
 - Fixed a build issue when compiling with `-fno-rtti`. This ensures compatibility with projects that disable
diff --git a/include/quill/bundled/fmt/base.h b/include/quill/bundled/fmt/base.h
index d4d55eab..03dedc42 100644
--- a/include/quill/bundled/fmt/base.h
+++ b/include/quill/bundled/fmt/base.h
@@ -1623,7 +1623,7 @@ struct type_is_unformattable_for {
 struct type_is_unformattable_for;
 #endif
 
-template <bool PACKED, typename Context, typename T, FMTQUILL_ENABLE_IF(PACKED)>
+template <bool QUILLPACKED, typename Context, typename T, FMTQUILL_ENABLE_IF(QUILLPACKED)>
 FMTQUILL_CONSTEXPR auto make_arg(T& val) -> value<Context> {
   using arg_type = remove_cvref_t<decltype(arg_mapper<Context>().map(val))>;
 
@@ -1662,7 +1662,7 @@ FMTQUILL_CONSTEXPR auto make_arg(T& val) -> basic_format_arg<Context> {
   return arg;
 }
 
-template <bool PACKED, typename Context, typename T, FMTQUILL_ENABLE_IF(!PACKED)>
+template <bool QUILLPACKED, typename Context, typename T, FMTQUILL_ENABLE_IF(!QUILLPACKED)>
 FMTQUILL_CONSTEXPR inline auto make_arg(T& val) -> basic_format_arg<Context> {
   return make_arg<Context>(val);
 }
diff --git a/include/quill/sinks/ConsoleSink.h b/include/quill/sinks/ConsoleSink.h
index 4d47318e..a0492022 100644
--- a/include/quill/sinks/ConsoleSink.h
+++ b/include/quill/sinks/ConsoleSink.h
@@ -170,7 +170,7 @@ class ConsoleColours
   }
 
 private:
-  std::array<WORD, 10> _colours = {0}; /**< Colours per log level */
+  std::array<WORD, 11> _colours = {0}; /**< Colours per log level */
   bool _using_colours{false};
   bool _can_use_colours{false};
 };
@@ -344,7 +344,7 @@ class ConsoleColours
   }
 
 private:
-  std::array<std::string_view, 10> _colours; /**< Colours per log level */
+  std::array<std::string_view, 11> _colours; /**< Colours per log level */
   bool _using_colours{false};
   bool _can_use_colours{false};
 };