SDL: Don't uselessly free the_host_buffer

It always points to a member of an SDL_Surface, which is freed by SDL.
This commit is contained in:
Dave Vasilevsky 2013-04-22 03:48:13 -04:00
parent c0d7aa0004
commit 94ebea3509

View File

@ -691,11 +691,6 @@ driver_base::~driver_base()
} }
#ifdef ENABLE_VOSF #ifdef ENABLE_VOSF
else { else {
if (the_host_buffer) {
D(bug(" freeing the_host_buffer at %p\n", the_host_buffer));
free(the_host_buffer);
the_host_buffer = NULL;
}
if (the_buffer_copy) { if (the_buffer_copy) {
D(bug(" freeing the_buffer_copy at %p\n", the_buffer_copy)); D(bug(" freeing the_buffer_copy at %p\n", the_buffer_copy));
free(the_buffer_copy); free(the_buffer_copy);
@ -817,10 +812,6 @@ driver_window::driver_window(SDL_monitor_desc &m)
// Close display // Close display
driver_window::~driver_window() driver_window::~driver_window()
{ {
#ifdef ENABLE_VOSF
if (use_vosf)
the_host_buffer = NULL; // don't free() in driver_base dtor
#endif
if (s) if (s)
SDL_FreeSurface(s); SDL_FreeSurface(s);
} }
@ -947,10 +938,6 @@ driver_fullscreen::driver_fullscreen(SDL_monitor_desc &m)
// Close display // Close display
driver_fullscreen::~driver_fullscreen() driver_fullscreen::~driver_fullscreen()
{ {
#ifdef ENABLE_VOSF
if (use_vosf)
the_host_buffer = NULL; // don't free() in driver_base dtor
#endif
if (s) if (s)
SDL_FreeSurface(s); SDL_FreeSurface(s);