use <fmt/core.h> instead of <fmt/format.h>

Should lighten compile times.

The only loss here is we'll no longer inherit e.g. "float" format specifiers for qvec3f.

common/bspfile.cc

@@ -25,7 +25,7 @@
 #include <cstdint>
 #include <limits.h>
 
-#include <fmt/format.h>
+#include <fmt/core.h>
 
 #include <atomic>
 #include <mutex>

common/decompile.cc

@@ -35,7 +35,7 @@
 #include <utility>
 #include <tuple>
 
-#include <fmt/format.h>
+#include <fmt/core.h>
 #include <fmt/ostream.h>
 
 #include "tbb/parallel_for.h"

common/qvec.cc

@@ -20,7 +20,7 @@
 #include <common/qvec.hh>
 
 #include <cmath> // for NAN
-#include <fmt/format.h>
+#include <fmt/core.h>
 
 /*
  * SGI FREE SOFTWARE LICENSE B (Version 2.0, Sept. 18, 2008)

include/common/bspfile_generic.hh

@@ -19,7 +19,7 @@
 
 #pragma once
 
-#include <fmt/format.h>
+#include <fmt/core.h>
 
 #include <cinttypes>
 #include <array>
@@ -271,7 +271,9 @@ enum class plane_type_t
 };
 
 // Fmt support
-template <> struct fmt::formatter<plane_type_t> : formatter<string_view> {
+template <> struct fmt::formatter<plane_type_t> {
+    constexpr auto parse(format_parse_context &ctx) -> decltype(ctx.begin()) { return ctx.end(); }
+
     template <typename FormatContext>
     auto format(plane_type_t t, FormatContext& ctx) {
         string_view name = "unknown";
@@ -284,7 +286,7 @@ template <> struct fmt::formatter<plane_type_t> : formatter<string_view> {
             case plane_type_t::PLANE_ANYY: name = "PLANE_ANYY"; break;
             case plane_type_t::PLANE_ANYZ: name = "PLANE_ANYZ"; break;
         }
-        return formatter<string_view>::format(name, ctx);
+        return format_to(ctx.out(), "{}", name);
     }
 };
 

include/common/fs.hh

@@ -131,15 +131,17 @@ inline fs::path DefaultExtension(const fs::path &path, const fs::path &extension
     return fs::path(path).replace_extension(extension);
 }
 
-#include <fmt/format.h>
+#include <fmt/core.h>
 
 // TODO: no wchar_t support in this version apparently
 template<>
-struct fmt::formatter<fs::path> : formatter<std::string>
+struct fmt::formatter<fs::path>
 {
+    constexpr auto parse(format_parse_context &ctx) -> decltype(ctx.begin()) { return ctx.end(); }
+
     template<typename FormatContext>
     auto format(const fs::path &p, FormatContext &ctx)
     {
-        return formatter<std::string>::format(p.string(), ctx);
+        return format_to(ctx.out(), "{}", p.string());
     }
 };

include/common/log.hh

@@ -30,7 +30,7 @@
 #include <cstdarg>
 #include <filesystem>
 #include <list>
-#include <fmt/format.h>
+#include <fmt/core.h>
 #include <common/bitflags.hh>
 #include <common/fs.hh>
 

include/common/qvec.hh

@@ -26,7 +26,7 @@
 #include <algorithm>
 #include <array>
 #include <ostream>
-#include <fmt/format.h>
+#include <fmt/core.h>
 #include <tuple>
 #include "common/mathlib.hh"
 #include "common/cmdlib.hh"
@@ -340,17 +340,19 @@ public:
 
 // Fmt support
 template<class T, size_t N>
-struct fmt::formatter<qvec<T, N>> : formatter<T>
+struct fmt::formatter<qvec<T, N>>
 {
+    constexpr auto parse(format_parse_context &ctx) -> decltype(ctx.begin()) { return ctx.end(); }
+
     template<typename FormatContext>
     auto format(const qvec<T, N> &p, FormatContext &ctx) -> decltype(ctx.out())
     {
         for (size_t i = 0; i < N - 1; i++) {
-            formatter<T>::format(p[i], ctx);
+            format_to(ctx.out(), "{}", p[i]);
             format_to(ctx.out(), " ");
         }
 
-        return formatter<T>::format(p[N - 1], ctx);
+        return format_to(ctx.out(), "{}", p[N - 1]);
     }
 };
 

lightpreview/glview.cpp

@@ -26,7 +26,7 @@ See file, 'COPYING', for details.
 #include <QWheelEvent>
 #include <QKeyEvent>
 #include <QTime>
-#include <fmt/format.h>
+#include <fmt/core.h>
 
 GLView::GLView(QWidget *parent)
     : QOpenGLWidget(parent), m_keysPressed(0), m_keymoveUpdateTimer(0), m_lastMouseDownPos(0, 0), m_displayAspect(1),

tests/test_light.cc

@@ -5,7 +5,6 @@
 
 #include <random>
 #include <algorithm> // for std::sort
-#include <fmt/format.h>
 
 #include <common/qvec.hh>