Skip to content

Commit 53bcec1

Browse files
Khakim-em
authored andcommitted
chore: fix C++ warning (#10922)
1 parent d6c683c commit 53bcec1

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -200,8 +200,8 @@ jobs:
200200
"test": true,
201201
// NOTE: `test-speedcenter` currently seems to be broken on `ubuntu-latest`
202202
"test-speedcenter": large && level >= 2,
203-
// made explicit until it can be assumed to have propagated to PRs
204-
"CMAKE_OPTIONS": "-DUSE_LAKE=ON",
203+
// We are not warning-free yet on all platforms, start here
204+
"CMAKE_OPTIONS": "-DLEAN_EXTRA_CXX_FLAGS=-Werror",
205205
},
206206
{
207207
"name": "Linux Reldebug",

src/util/timeit.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,7 @@ class xtimeit {
6969
m_start = std::move(other.m_start);
7070
m_fn = std::move(other.m_fn); // TODO: use `std::exchange(_, nullptr)` in C++14
7171
other.m_fn = nullptr;
72+
return *this;
7273
}
7374
~xtimeit() {
7475
if (!m_fn) return;

0 commit comments

Comments
 (0)