Fixed memory leaks.
authorTeemu Murtola <teemu.murtola@gmail.com>
Fri, 16 Dec 2011 18:00:10 +0000 (20:00 +0200)
committerTeemu Murtola <teemu.murtola@gmail.com>
Sat, 17 Dec 2011 07:12:39 +0000 (09:12 +0200)
commitc5b40970a7a86cfc0515e6c49e86b79487795592
tree77b74e6fdd821280e17880403f8011b7b624e1c2
parent74168913f67bbebb174dc5c2c522bfd0c66ccb50
Fixed memory leaks.

Fixed two memory leaks:
 - AnalysisData leaked memory if startData() was called more than once.
 - Memory for the error collector object in selection parser was not
   properly managed.  Now the object is no longer dynamically allocated,
   which is much cleaner.

Also added a call to xmlCleanupParser() at the end of tests to remove
static memory allocated by libxml2 from, e.g., valgrind reports.

Also added suppressions for legacy and external library code.

Change-Id: I3ec213b6441d38f46bda138c959dbb5990144027
12 files changed:
CMakeLists.txt
admin/GerritBuild
cmake/legacy_and_external.supp [new file with mode: 0644]
src/gromacs/analysisdata/analysisdata.cpp
src/gromacs/selection/scanner.h
src/gromacs/selection/scanner_internal.cpp
src/gromacs/selection/selectioncollection.cpp
src/testutils/CMakeLists.txt
src/testutils/TestMacros.cmake
src/testutils/refdata-common.cpp
src/testutils/refdata.cpp
src/testutils/refdata.h