From fa633dc717422aec4e76d2c5ebc8fe70ce668d14 Mon Sep 17 00:00:00 2001
From: Chris Danford <chrisdanford@users.sourceforge.net>
Date: Fri, 29 Aug 2003 04:34:40 +0000
Subject: [PATCH] fix crash in course editor

---
 stepmania/src/EditCoursesMenu.cpp | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/stepmania/src/EditCoursesMenu.cpp b/stepmania/src/EditCoursesMenu.cpp
index 1e8bb5f51a..6988cf5550 100644
--- a/stepmania/src/EditCoursesMenu.cpp
+++ b/stepmania/src/EditCoursesMenu.cpp
@@ -478,14 +478,6 @@ void EditCoursesMenu::OnRowValueChanged( Row row )
 	Course* pCourse = GetSelectedCourse();
 	CourseEntry* pEntry = GetSelectedEntry();
 
-	// HACK:  I can't decide how to handle Courses with 0 entries.  For now,
-	// just add one entry to it.
-	if( pEntry == NULL )
-	{
-		pCourse->m_entries.push_back( CourseEntry() );
-		pEntry = GetSelectedEntry();
-	}
-
 	switch( row )
 	{
 	case ROW_COURSE:
@@ -494,6 +486,12 @@ void EditCoursesMenu::OnRowValueChanged( Row row )
 		m_CourseBanner.LoadFromCourse( pCourse );
 		m_CourseBanner.ScaleToClipped( COURSE_BANNER_WIDTH, COURSE_BANNER_HEIGHT );
 		m_iSelection[ROW_ENTRY] = 0;
+		pEntry = GetSelectedEntry();
+		if( pEntry == NULL )
+		{
+			pCourse->m_entries.push_back( CourseEntry() );
+			pEntry = GetSelectedEntry();
+		}
 		// fall through
 	case ROW_COURSE_OPTIONS:
 		CHECKPOINT;