Merge pull request #27 from lxylxy123456/hire_assistant

Fix memory leak in src/HireAssistantTest.cpp

Makefile

@@ -34,7 +34,6 @@ VALGRIND := $(filter-out valgrind/RaceExampleTest,$(VALGRIND))
 # VALGRIND_ALL has currently unavailable tests filtered out
 VALGRIND_ALL := $(VALGRIND)
 VALGRIND_ALL := $(filter-out valgrind/FibTest,$(VALGRIND_ALL))
-VALGRIND_ALL := $(filter-out valgrind/HireAssistantTest,$(VALGRIND_ALL))
 VALGRIND_ALL := $(filter-out valgrind/HuffmanTest,$(VALGRIND_ALL))
 VALGRIND_ALL := $(filter-out valgrind/IntervalTreeTest,$(VALGRIND_ALL))
 VALGRIND_ALL := $(filter-out valgrind/MatVecTest,$(VALGRIND_ALL))

src/HireAssistantMain.cpp

@@ -38,5 +38,8 @@ int main(int argc, char *argv[]) {
 		candidates.push_back(new Candidate<int>(*i, ci, ch, false));
 	output_integers(abilities);
 	std::cout << HireAssistant(candidates) << std::endl;
+	for (std::vector<Candidate<int>*>::iterator i = candidates.begin();
+		i < candidates.end(); i++)
+		delete *i;
 	return 0;
 }

src/HireAssistantTest.cpp

@@ -41,6 +41,9 @@ int test(int n, int ci, int ch) {
 		assert(ans > 0);
 	else
 		assert(ans == 0);
+	for (std::vector<Candidate<int>*>::iterator i = candidates.begin();
+		i < candidates.end(); i++)
+		delete *i;
 	return 0;
 }