From 55ad6158a6474d13d3ecaff38ccd1b2e48e16346 Mon Sep 17 00:00:00 2001
From: Glenn Maynard <gmaynard@users.sourceforge.net>
Date: Fri, 23 Aug 2002 06:50:48 +0000
Subject: [PATCH] Fix a memleak that doesn't matter

(still havn't found the memleak I'm looking for, but I'll find every harmless
one instead!)
---
 stepmania/src/ScreenManager.cpp | 19 +++++++++++--------
 stepmania/src/ScreenManager.h   |  2 ++
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/stepmania/src/ScreenManager.cpp b/stepmania/src/ScreenManager.cpp
index 4b3b317368..ba23368cfc 100644
--- a/stepmania/src/ScreenManager.cpp
+++ b/stepmania/src/ScreenManager.cpp
@@ -77,13 +77,21 @@ ScreenManager::~ScreenManager()
 {
 	LOG->Trace( "ScreenManager::~ScreenManager()" );
 
+	EmptyDeleteQueue();
+
 	// delete current states
 	for( int i=0; i<m_ScreenStack.GetSize(); i++ )
 		SAFE_DELETE( m_ScreenStack[i] );
 }
 
+void ScreenManager::EmptyDeleteQueue()
+{
+	// delete all ScreensToDelete
+	for( int i=0; i<m_ScreensToDelete.GetSize(); i++ )
+		SAFE_DELETE( m_ScreensToDelete[i] );
 
-
+	m_ScreensToDelete.RemoveAll();
+}
 
 void ScreenManager::Update( float fDeltaTime )
 {
@@ -92,15 +100,10 @@ void ScreenManager::Update( float fDeltaTime )
 	for( int p=0; p<NUM_PLAYERS; p++ )
 		m_textCreditInfo[p].Update( fDeltaTime );
 
-	// delete all ScreensToDelete
-	int i;
-	for( i=0; i<m_ScreensToDelete.GetSize(); i++ )
-		SAFE_DELETE( m_ScreensToDelete[i] );
-
-	m_ScreensToDelete.RemoveAll();
+	EmptyDeleteQueue();
 
 	// Update all windows in the stack
-	for( i=0; i<m_ScreenStack.GetSize(); i++ )
+	for( int i=0; i<m_ScreenStack.GetSize(); i++ )
 		m_ScreenStack[i]->Update( fDeltaTime );
 
 }
diff --git a/stepmania/src/ScreenManager.h b/stepmania/src/ScreenManager.h
index 971f8978c1..53b8dce273 100644
--- a/stepmania/src/ScreenManager.h
+++ b/stepmania/src/ScreenManager.h
@@ -50,6 +50,8 @@ private:
 	BitmapText m_textStats;
 	BitmapText m_textSystemMessage;
 	BitmapText m_textCreditInfo[NUM_PLAYERS];
+
+	void EmptyDeleteQueue();
 };