[ros-diffs] [hbelusca] 75427: [ATL]: - Both the CAtlList class and its helper class CNode do not support copy construction. - Remove a redundant call to GetFreeNode(). CORE-13597

Author: hbelusca
Date: Thu Jul 27 19:08:55 2017
New Revision: 75427

URL: http://svn.reactos.org/svn/reactos?rev=75427&view=rev
Log:
[ATL]: - Both the CAtlList class and its helper class CNode do not support copy construction. - Remove a redundant call to GetFreeNode().
CORE-13597

Modified:
    trunk/reactos/sdk/lib/atl/atlcoll.h

Modified: trunk/reactos/sdk/lib/atl/atlcoll.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/sdk/lib/atl/atlcoll.h?rev=7...
==============================================================================
--- trunk/reactos/sdk/lib/atl/atlcoll.h	[iso-8859-1] (original)
+++ trunk/reactos/sdk/lib/atl/atlcoll.h	[iso-8859-1] Thu Jul 27 19:08:55 2017
@@ -155,6 +155,11 @@
             m_Element(Element)
         {
         }
+
+    /* The CNode class does not support construction by copy */
+    private:
+        CNode(_In_ const CNode&);
+        CNode& operator=(_In_ const CNode&);
     };

private:
@@ -165,6 +170,11 @@
     CNode* m_FreeNode;
     size_t m_NumElements;

+/* The CAtlList class does not support construction by copy */
+private:
+    CAtlList(_In_ const CAtlList&);
+    CAtlList& operator=(_In_ const CAtlList&);
+
 public:
     CAtlList(_In_ UINT nBlockSize = 10);
     ~CAtlList();
@@ -481,7 +491,7 @@
 {
     GetFreeNode();

-    CNode* NewNode = GetFreeNode();
+    CNode* NewNode = m_FreeNode;
     CNode* NextFree = m_FreeNode->m_Next;

NewNode = new CNode(element);