fable,runtime: Use fmt::runtime() to ensure C++20 compatibility

Author: cassava

fable/include/fable/conf.hpp

@@ -332,7 +332,7 @@ class Conf {
 
   template <typename... Args>
   [[noreturn]] void throw_error(std::string_view format, Args&&... args) const {
-    throw_error(fmt::format(format, std::forward<Args>(args)...));
+    throw_error(fmt::format(fmt::runtime(format), std::forward<Args>(args)...));
   }
   [[noreturn]] void throw_error(const std::string& msg) const;
   [[noreturn]] void throw_unexpected(const std::string& key, const std::string& msg = "") const;

fable/include/fable/error.hpp

@@ -52,7 +52,7 @@ class Error : public std::exception {
 
   template <typename... Args>
   explicit Error(std::string_view format, Args&&... args)
-      : message_(fmt::format(format, std::forward<Args>(args)...)) {}
+      : message_(fmt::format(fmt::runtime(format), std::forward<Args>(args)...)) {}
 
   [[nodiscard]] const char* what() const noexcept override { return message_.c_str(); }
 };

runtime/include/cloe/core/error.hpp

@@ -36,7 +36,8 @@ class Error : public std::exception {
   explicit Error(const char* what) : err_(what) {}
 
   template <typename... Args>
-  explicit Error(const char* format, const Args&... args) : err_(fmt::format(format, args...)) {}
+  Error(std::string_view format, Args&&... args)
+      : err_(fmt::format(fmt::runtime(format), std::forward<Args>(args)...)) {}
 
   virtual ~Error() noexcept = default;
 
@@ -47,8 +48,8 @@ class Error : public std::exception {
   void set_explanation(const std::string& explanation);
 
   template <typename... Args>
-  void set_explanation(const char* format, const Args&... args) {
-    set_explanation(fmt::format(format, args...));
+  void set_explanation(std::string_view format, Args&&... args) {
+    set_explanation(fmt::format(fmt::runtime(format), std::forward<Args>(args)...));
   }
 
   const std::string& explanation() const { return explanation_; }
@@ -59,8 +60,8 @@ class Error : public std::exception {
   }
 
   template <typename... Args>
-  Error explanation(const char* format, const Args&... args) && {
-    set_explanation(fmt::format(format, args...));
+  Error explanation(std::string_view format, Args&&... args) && {
+    set_explanation(fmt::format(fmt::runtime(format), std::forward<Args>(args)...));
     return std::move(*this);
   }
 

runtime/include/cloe/model.hpp

@@ -75,8 +75,8 @@ class ModelError : public Error {
   }
 
   template <typename... Args>
-  ModelError explanation(const char* format, const Args&... args) && {
-    this->set_explanation(fmt::format(format, args...));
+  ModelError explanation(std::string_view format, const Args&... args) && {
+    this->set_explanation(fmt::format(fmt::runtime(format), args...));
     return std::move(*this);
   }
 };
@@ -100,8 +100,8 @@ class ModelAbort : public ModelError {
   }
 
   template <typename... Args>
-  ModelError explanation(const char* format, const Args&... args) && {
-    this->set_explanation(fmt::format(format, args...));
+  ModelAbort explanation(std::string_view format, const Args&... args) && {
+    this->set_explanation(fmt::format(fmt::runtime(format), args...));
     return std::move(*this);
   }
 };
@@ -123,8 +123,8 @@ class ModelReset : public ModelError {
   }
 
   template <typename... Args>
-  ModelError explanation(const char* format, const Args&... args) && {
-    this->set_explanation(fmt::format(format, args...));
+  ModelReset explanation(std::string_view format, const Args&... args) && {
+    this->set_explanation(fmt::format(fmt::runtime(format), args...));
     return std::move(*this);
   }
 };
@@ -146,8 +146,8 @@ class ModelStop : public ModelError {
   }
 
   template <typename... Args>
-  ModelError explanation(const char* format, const Args&... args) && {
-    this->set_explanation(fmt::format(format, args...));
+  ModelStop explanation(std::string_view format, const Args&... args) && {
+    this->set_explanation(fmt::format(fmt::runtime(format), args...));
     return std::move(*this);
   }
 };