Surface: Use the destructor as intended.

There is no reason to have an explicit free() method instead of just
destructing the Surface object and doing cleanup in the destructor.

src/gmenu2x.cpp

@@ -327,7 +327,7 @@ GMenu2X::~GMenu2X() {
 void GMenu2X::quit() {
 	fflush(NULL);
 	sc.clear();
-	s->free();
+	free(s);
 	SDL_Quit();
 #ifdef TARGET_GP2X
 /*	if (gp2x_mem!=0) {

src/surface.cpp

@@ -66,10 +66,6 @@ Surface::Surface(const string &img, const string &skin) {
 }
 
 Surface::~Surface() {
-	free();
-}
-
-void Surface::free() {
 	if (freeWhenDone) {
 		SDL_FreeSurface(raw);
 	}
@@ -83,8 +79,6 @@ SDL_PixelFormat *Surface::format() {
 }
 
 void Surface::load(const string &img, const string &skin) {
-	free();
-
 	string skinpath;
 	if (!skin.empty() && !img.empty() && img[0]!='/') {
 		skinpath = "skins/"+skin+"/"+img;

src/surface.h

@@ -47,7 +47,6 @@ public:
 
 	SDL_Surface *raw;
 
-	void free();
 	void flip();
 
 	void clearClipRect();

src/surfacecollection.cpp

@@ -109,11 +109,7 @@ void SurfaceCollection::del(const string &path) {
 }
 
 void SurfaceCollection::clear() {
-	while (surfaces.size()>0) {
+	surfaces.clear();
-		surfaces.begin()->second->free();
-		free(surfaces.begin()->second);
-		surfaces.erase(surfaces.begin());
-	}
 }
 
 void SurfaceCollection::move(const string &from, const string &to) {