Fixed a bug in cdhashmap in which doubly-linked list was not properly cleaned up on a call to obliterate.

Also, removed some experimental code and a unit test from cdmap_black that used it. This test created a CDList *in* context memory which seems like a very bad idea (and
it was improperly implemented resulting in a memory leak).

1 files changed, 0 insertions, 17 deletions

diff --git a/src/context/cdlist.h b/src/context/cdlist.h
index 4a5ebfd30..47667cc8d 100644
--- a/src/context/cdlist.h
+++ b/src/context/cdlist.h
@@ -238,23 +238,6 @@ public:
   }
 
   /**
-   * Main constructor: d_list starts as NULL, size is 0
-   */
-  CDList(bool allocatedInCMM,
-            Context* context,
-            bool callDestructor = true,
-            const CleanUp& cleanup = CleanUp(),
-            const Allocator& alloc = Allocator()) :
-    ContextObj(allocatedInCMM, context),
-    d_list(NULL),
-    d_size(0),
-    d_callDestructor(callDestructor),
-    d_sizeAlloc(0),
-    d_cleanUp(cleanup),
-    d_allocator(alloc) {
-  }
-
-  /**
    * Destructor: delete the list
    */
   ~CDList() {