IterableList: fixed a complex bug that can be called "ABA problem of null pointer"

[libcds.git] / cds / container / details / make_split_list_set_iterable_list.h

diff --git a/cds/container/details/make_split_list_set_iterable_list.h b/cds/container/details/make_split_list_set_iterable_list.h
index f38edb5e3c7f40bbaa8d9bba5af1e92d776800e5..87e511e4a02823624192ee90ae29f47d2efac28a 100644 (file)
--- a/cds/container/details/make_split_list_set_iterable_list.h
+++ b/cds/container/details/make_split_list_set_iterable_list.h
@@ -51,9 +51,10 @@ namespace cds { namespace container { namespace details {
             value_type  m_Value;
 
             template <typename Q>
-            explicit node_type( Q const& v )
-                : m_Value(v)
+            explicit node_type( Q&& v )
+                : m_Value( std::forward<Q>( v ))
             {}
+
             template <typename Q, typename... Args>
             explicit node_type( Q&& q, Args&&... args )
                 : m_Value( std::forward<Q>(q), std::forward<Args>(args)... )
@@ -116,6 +117,7 @@ namespace cds { namespace container { namespace details {
                 {
                     return base_class::operator()( key_accessor()( v.m_Value ));
                 }
+
                 template <typename Q>
                 size_t operator()( Q const& k ) const
                 {