Memtables should be deleted appropriately in the unit test.

Summary: Memtables should be deleted appropriately in the unit test. Test Plan: Reviewers: CC: Task ID: # Blame Rev:
vinniefalco · Dec 2, 2013 · 96bc3ec · 96bc3ec
1 parent 45a2f2d
commit 96bc3ec
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/db/write_batch_test.cc b/db/write_batch_test.cc
@@ -69,7 +69,7 @@ static std::string PrintContents(WriteBatch* b) {
   } else if (count != WriteBatchInternal::Count(b)) {
     state.append("CountMismatch()");
   }
-  mem->Unref();
+  delete mem->Unref();
   return state;
 }
 

diff --git a/table/table_test.cc b/table/table_test.cc
@@ -374,10 +374,10 @@ class MemTableConstructor: public Constructor {
     memtable_->Ref();
   }
   ~MemTableConstructor() {
-    memtable_->Unref();
+    delete memtable_->Unref();
   }
   virtual Status FinishImpl(const Options& options, const KVMap& data) {
-    memtable_->Unref();
+    delete memtable_->Unref();
     memtable_ = new MemTable(internal_comparator_, table_factory_);
     memtable_->Ref();
     int seq = 1;
@@ -1289,7 +1289,7 @@ TEST(MemTableTest, Simple) {
   }
 
   delete iter;
-  memtable->Unref();
+  delete memtable->Unref();
 }