From dec007693a6a0e097623158a95642cc78cdff97f Mon Sep 17 00:00:00 2001
From: Sebastian Ullrich <sebasti@nullri.ch>
Date: Fri, 24 Oct 2025 13:09:08 +0200
Subject: [PATCH] chore: fix C++ warning (#10922)

---
 .github/workflows/ci.yml | 4 ++--
 src/util/timeit.h        | 1 +
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d4ce117fcb..af266d6573 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -200,8 +200,8 @@ jobs:
                 "test": true,
                 // NOTE: `test-speedcenter` currently seems to be broken on `ubuntu-latest`
                 "test-speedcenter": large && level >= 2,
-                // made explicit until it can be assumed to have propagated to PRs
-                "CMAKE_OPTIONS": "-DUSE_LAKE=ON",
+                // We are not warning-free yet on all platforms, start here
+                "CMAKE_OPTIONS": "-DLEAN_EXTRA_CXX_FLAGS=-Werror",
               },
               {
                 "name": "Linux Reldebug",
diff --git a/src/util/timeit.h b/src/util/timeit.h
index ef3237d1c7..954b8a0637 100644
--- a/src/util/timeit.h
+++ b/src/util/timeit.h
@@ -69,6 +69,7 @@ public:
         m_start     = std::move(other.m_start);
         m_fn        = std::move(other.m_fn); // TODO: use `std::exchange(_, nullptr)` in C++14
         other.m_fn = nullptr;
+        return *this;
     }
     ~xtimeit() {
         if (!m_fn) return;