Made unique_resource::reset() implementation slightly more efficient.

boostorg · Jan 28, 2024 · 6f2adf3 · 6f2adf3
1 parent 7be5037
commit 6f2adf3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/include/boost/scope/unique_resource.hpp b/include/boost/scope/unique_resource.hpp
@@ -1402,7 +1402,7 @@ class unique_resource
     {
         reset_impl
         (
-            static_cast< typename detail::move_or_copy_assign_ref< R, resource_type >::type >(res),
+            static_cast< R&& >(res),
             typename detail::conjunction<
                 detail::is_nothrow_invocable< deleter_type&, resource_type& >,
                 std::is_nothrow_assignable< internal_resource_type&, typename detail::move_or_copy_assign_ref< R, resource_type >::type >
@@ -1508,7 +1508,7 @@ class unique_resource
     void reset_impl(R&& res, std::true_type) noexcept
     {
         reset();
-        m_data.assign_resource(static_cast< R&& >(res));
+        m_data.assign_resource(static_cast< typename detail::move_or_copy_assign_ref< R, resource_type >::type >(res));
     }
 
     //! Assigns a new resource object to the unique resource wrapper.
@@ -1518,7 +1518,7 @@ class unique_resource
         try
         {
             reset();
-            m_data.assign_resource(static_cast< R&& >(res));
+            m_data.assign_resource(static_cast< typename detail::move_or_copy_assign_ref< R, resource_type >::type >(res));
         }
         catch (...)
         {