refactor(lock): simplify assign constraint

* This fixes a bug in Clang >= 17, see: llvm/llvm-project#74447
Curve · Dec 29, 2023 · 50bc274 · 50bc274
1 parent c3a0063
commit 50bc274
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/include/lockpp/lock.hpp b/include/lockpp/lock.hpp
@@ -40,7 +40,7 @@ namespace lockpp
       public:
         template <typename O>
             void assign(O &&value) &
-                requires std::assignable_from<Type &, decltype(std::forward<O>(value))>;
+                requires std::assignable_from<Type &, O>;
 
       public:
         [[nodiscard]] Type &get_unsafe() &;

diff --git a/include/lockpp/lock.inl b/include/lockpp/lock.inl
@@ -26,7 +26,7 @@ namespace lockpp
     template <typename Type, decayed Mutex>
         template <typename O>
         void lock<Type, Mutex>::assign(O &&value) &
-            requires std::assignable_from<Type &, decltype(std::forward<O>(value))>
+            requires std::assignable_from<Type &, O>
     {
         auto locked = write();
         *locked = std::forward<O>(value);