don't do SG_FREE on varray.begin

This is now handled in the destructor of v_array and additional free's only cause double free bugs.
shogun-toolbox · Apr 27, 2012 · 4574712 · 4574712
1 parent 0afeee4
commit 4574712
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 20 deletions.
diff --git a/src/shogun/classifier/vw/VwParser.cpp b/src/shogun/classifier/vw/VwParser.cpp
@@ -41,13 +41,6 @@ CVwParser::CVwParser(CVwEnvironment* env_to_use)
 
 CVwParser::~CVwParser()
 {
-	SG_FREE(channels.begin);
-	channels.begin = channels.end = channels.end_array = NULL;
-	SG_FREE(words.begin);
-	words.begin = words.end = words.end_array = NULL;
-	SG_FREE(name.begin);
-	name.begin = name.end = name.end_array = NULL;
-
 	SG_UNREF(env);
 	SG_UNREF(cache_writer);
 }

diff --git a/src/shogun/classifier/vw/vw_example.cpp b/src/shogun/classifier/vw/vw_example.cpp
@@ -29,18 +29,6 @@ VwExample::~VwExample()
 {
 	if (ld)
 		delete ld;
-	if (tag.end_array != tag.begin)
-	{
-		SG_FREE(tag.begin);
-		tag.end_array = tag.begin;
-	}
-
-	for (vw_size_t j = 0; j < 256; j++)
-	{
-		if (atomics[j].begin != atomics[j].end_array)
-			SG_FREE(atomics[j].begin);
-	}
-	SG_FREE(indices.begin);
 }
 
 void VwExample::reset_members()

diff --git a/src/shogun/io/IOBuffer.cpp b/src/shogun/io/IOBuffer.cpp
@@ -31,7 +31,6 @@ CIOBuffer::CIOBuffer(int fd)
 
 CIOBuffer::~CIOBuffer()
 {
-	free(space.begin);
 }
 
 void CIOBuffer::init()