Make redundant ctors for clang-12

arbor-sim · Sep 25, 2024 · 3095381 · 3095381
1 parent bba2dd6
commit 3095381
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/.github/workflows/sanitize.yml b/.github/workflows/sanitize.yml
@@ -19,7 +19,7 @@ permissions:
 jobs:
  build:
  name: "Sanitize"
- runs-on: ubuntu-22.04
+ runs-on: ubuntu-24.04
  strategy:
  fail-fast: false
  matrix:

diff --git a/arbor/cv_policy.cpp b/arbor/cv_policy.cpp
@@ -40,7 +40,8 @@ struct cvp_cv_policy_plus {
  return os;
  }
 
- cvp_cv_policy_plus(const cv_policy& lhs, const cv_policy& rhs): lhs_{lhs}, rhs_(rhs) {}
+ // TODO This is needed seemingly only for older compilers
+ cvp_cv_policy_plus(cv_policy lhs, cv_policy rhs): lhs_{std::move(lhs)}, rhs_(std::move(rhs)) {}
 
  cv_policy lhs_, rhs_;
 };
@@ -63,6 +64,9 @@ struct cvp_cv_policy_bar {
  return os;
  }
 
+ // TODO This is needed seemingly only for older compilers
+ cvp_cv_policy_bar(cv_policy lhs, cv_policy rhs): lhs_{std::move(lhs)}, rhs_(std::move(rhs)) {}
+
  cv_policy lhs_, rhs_;
 };