From 7ddd4ff02e29ec06bf53009144ee4bb813187bed Mon Sep 17 00:00:00 2001 From: JJ Marr Date: Mon, 23 Sep 2024 10:04:24 -0400 Subject: [PATCH] Allow partial moving The official C++ core guidelines policy is you must move rvalues. This is inflexible in practice in the Wesnoth codebase. --- .clang-tidy | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.clang-tidy b/.clang-tidy index cdd6be69cf5..719507dac6f 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,4 +1,7 @@ --- Checks: '-*,bugprone-move-forwarding-reference,bugprone-use-after-move,cppcoreguidelines-rvalue-reference-param-not-moved,modernize-use-nullptr,performance-move-const-arg,performance-move-constructor-init,performance-no-automatic-move,performance-unnecessary-value-param' +CheckOptions: + - key: cppcoreguidelines-rvalue-reference-param-not-moved.AllowPartialMove + value: true WarningsAsErrors: true ...