Merge pull request #26 from lxylxy123456/dynamic_table

Fix memory leak in DynamicTable
lxylxy123456 · Dec 17, 2020 · 3af51bb · 3af51bb
2 parents effa536 + 2dadc60
commit 3af51bb
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/Makefile b/Makefile
@@ -33,7 +33,6 @@ VALGRIND := $(filter-out valgrind/RaceExampleTest,$(VALGRIND))
 
 # VALGRIND_ALL has currently unavailable tests filtered out
 VALGRIND_ALL := $(VALGRIND)
-VALGRIND_ALL := $(filter-out valgrind/DynamicTableTest,$(VALGRIND_ALL))
 VALGRIND_ALL := $(filter-out valgrind/FibTest,$(VALGRIND_ALL))
 VALGRIND_ALL := $(filter-out valgrind/HireAssistantTest,$(VALGRIND_ALL))
 VALGRIND_ALL := $(filter-out valgrind/HuffmanTest,$(VALGRIND_ALL))

diff --git a/include/DynamicTable.hpp b/include/DynamicTable.hpp
@@ -59,6 +59,7 @@ class DynamicTable {
 			}
 			return ans;
 		}
+		~DynamicTable() { delete table; }
 		size_t size, num;
 		std::vector<T>* table;
 };