From afde724122ed064835a609dad793147d24a1dc3e Mon Sep 17 00:00:00 2001
From: Glenn Maynard <gmaynard@users.sourceforge.net>
Date: Tue, 15 Jul 2003 19:41:13 +0000
Subject: [PATCH] fix silly BMS loading bugs

---
 stepmania/src/NotesLoaderBMS.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/stepmania/src/NotesLoaderBMS.cpp b/stepmania/src/NotesLoaderBMS.cpp
index 51bdf28113..7a99995f35 100644
--- a/stepmania/src/NotesLoaderBMS.cpp
+++ b/stepmania/src/NotesLoaderBMS.cpp
@@ -194,7 +194,7 @@ bool BMSLoader::LoadFromBMSFile( const CString &sPath, Notes &out )
 	}
 
 	// we're done reading in all of the BMS values
-	if( out.m_NotesType )
+	if( out.m_NotesType == NOTES_TYPE_INVALID )
 	{
 		LOG->Warn("Couldn't determine note types of file '%s'", sPath.c_str() );
 		delete pNoteData;
@@ -275,9 +275,8 @@ bool BMSLoader::LoadFromDir( CString sDir, Song &out )
 	{
 		Notes* pNewNotes = new Notes;
 
-		LoadFromBMSFile( out.GetSongDir() + arrayBMSFileNames[i],
-			*pNewNotes );
-		if(pNewNotes->m_NotesType != NOTES_TYPE_INVALID)
+		const bool ok = LoadFromBMSFile( out.GetSongDir() + arrayBMSFileNames[i], *pNewNotes );
+		if( ok )
 			out.m_apNotes.push_back( pNewNotes );
 		else
 			delete pNewNotes;