From 9434a66d94745bd760ec7c06cf5ea8f2f7a3a6f1 Mon Sep 17 00:00:00 2001
From: Chris Danford <chrisdanford@users.sourceforge.net>
Date: Sun, 21 Sep 2003 23:16:44 +0000
Subject: [PATCH] fix FixImpossibleRows() fix misaligned editor NoteFields

---
 stepmania/src/NoteData.cpp     | 22 +++++++++++
 stepmania/src/NoteData.h       |  3 +-
 stepmania/src/NoteDataUtil.cpp | 48 ++++++++++++++++--------
 stepmania/src/NoteDataUtil.h   |  3 --
 stepmania/src/ScreenEdit.cpp   | 67 ++++++++++++++++++----------------
 stepmania/src/Song.cpp         | 20 ++++++----
 stepmania/src/Steps.cpp        |  2 +-
 7 files changed, 104 insertions(+), 61 deletions(-)

diff --git a/stepmania/src/NoteData.cpp b/stepmania/src/NoteData.cpp
index cb6ec03b66..577b84b185 100644
--- a/stepmania/src/NoteData.cpp
+++ b/stepmania/src/NoteData.cpp
@@ -521,3 +521,25 @@ void NoteData::SetTapNote(int track, int row, TapNote t)
 	PadTapNotes(row);
 	m_TapNotes[track][row]=t;
 }
+
+void NoteData::EliminateAllButOneTap(int row)
+{
+	if(row < 0) return;
+
+	PadTapNotes(row);
+
+	for(int track = 0; track < m_iNumTracks; ++track)
+	{
+		if( m_TapNotes[track][row] == TAP_TAP )
+			break;
+	}
+
+	track++;
+
+	for( ; track < m_iNumTracks; ++track)
+	{
+		if( m_TapNotes[track][row] == TAP_TAP )
+			m_TapNotes[track][row] = TAP_EMPTY;
+	}
+}
+
diff --git a/stepmania/src/NoteData.h b/stepmania/src/NoteData.h
index 7a05028bce..3de7f7e1f8 100644
--- a/stepmania/src/NoteData.h
+++ b/stepmania/src/NoteData.h
@@ -137,8 +137,7 @@ public:
 	// True if no notes in row that aren't true in the mask
 	bool RowPassesValidMask( int row, bool bValidMask[] );
 
-	// Remove all tap notes that fail this mask
-	void EliminateNonPassingTaps( int row, bool bValidMask[] ); 
+	void EliminateAllButOneTap( int row ); 
 };
 
 
diff --git a/stepmania/src/NoteDataUtil.cpp b/stepmania/src/NoteDataUtil.cpp
index 9d2907234e..d5accc092f 100644
--- a/stepmania/src/NoteDataUtil.cpp
+++ b/stepmania/src/NoteDataUtil.cpp
@@ -916,35 +916,53 @@ const ValidRow g_ValidRows[] =
 
 void NoteDataUtil::FixImpossibleRows( NoteData &in, StepsType st )
 {
-	for( unsigned i=0; i<ARRAYSIZE(g_ValidRows); i++ )
+	vector<const ValidRow*> vpValidRowsToCheck;
+	for( int i=0; i<ARRAYSIZE(g_ValidRows); i++ )
 	{
-		const ValidRow vr = g_ValidRows[i];
-		if( vr.st != st )
-			continue;	// skip
+		if( g_ValidRows[i].st == st )
+			vpValidRowsToCheck.push_back( &g_ValidRows[i] );
+	}
 
-		for( int r=0; r<=in.GetLastRow(); r++ )
-			if( !NoteDataUtil::RowPassesValidMask(in,r,vr.bValidMask) )	// failed mask
-				NoteDataUtil::EliminateNonPassingTaps(in,r,vr.bValidMask);
+	// bail early if there's nothing to validate against
+	if( vpValidRowsToCheck.empty() )
+		return;
+
+	// each row must pass at least one valid mask
+	for( int r=0; r<=in.GetLastRow(); r++ )
+	{
+		// only check rows with jumps
+		if( in.GetNumTapNonEmptyTracks(r) < 2 )
+			continue;
+
+		bool bPassedOneMask = false;
+		for( unsigned i=0; i<vpValidRowsToCheck.size(); i++ )
+		{
+			const ValidRow &vr = *vpValidRowsToCheck[i];
+			if( NoteDataUtil::RowPassesValidMask(in,r,vr.bValidMask) )
+			{
+				bPassedOneMask = true;
+				break;
+			}
+			else
+				int skjdksjd = 0;
+		}
+
+		if( !bPassedOneMask )
+			in.EliminateAllButOneTap(r);
 	}
 }
 
 bool NoteDataUtil::RowPassesValidMask( NoteData &in, int row, const bool bValidMask[] )
 {
 	for( int t=0; t<in.GetNumTracks(); t++ )
+	{
 		if( !bValidMask[t] && in.GetTapNote(t,row) != TAP_EMPTY )
 			return false;
+	}
 
 	return true;
 }
 
-void NoteDataUtil::EliminateNonPassingTaps( NoteData &in, int row, const bool bValidMask[] )
-{
-	for( int t=0; t<in.GetNumTracks(); t++ )
-		if( !bValidMask[t] && in.GetTapNote(t,row) != TAP_EMPTY )
-			in.SetTapNote(t,row,TAP_EMPTY);
-}
-
-
 void NoteDataUtil::ConvertAdditionsToRegular( NoteData &in )
 {
 	for( int r=0; r<=in.GetLastRow(); r++ )
diff --git a/stepmania/src/NoteDataUtil.h b/stepmania/src/NoteDataUtil.h
index f413cadd76..65ebca5c14 100644
--- a/stepmania/src/NoteDataUtil.h
+++ b/stepmania/src/NoteDataUtil.h
@@ -73,9 +73,6 @@ namespace NoteDataUtil
 	// True if no notes in row that aren't true in the mask
 	bool RowPassesValidMask( NoteData &in, int row, const bool bValidMask[] );
 
-	// Remove all tap notes that fail this mask
-	void EliminateNonPassingTaps( NoteData &in, int row, const bool bValidMask[] ); 
-
 	void TransformNoteData( NoteData &nd, PlayerOptions &po, StepsType st );
 
 	void Scale( NoteData &nd, float fScale );
diff --git a/stepmania/src/ScreenEdit.cpp b/stepmania/src/ScreenEdit.cpp
index 730d3d178a..4dea48fbef 100644
--- a/stepmania/src/ScreenEdit.cpp
+++ b/stepmania/src/ScreenEdit.cpp
@@ -42,41 +42,42 @@ const float RECORD_HOLD_SECONDS = 0.3f;
 // Defines specific to GameScreenTitleMenu
 //
 
-const float MAX_SECONDS_CAN_BE_OFF_BY	=	0.20f;
-const float GRAY_ARROW_Y				= ARROW_SIZE * 1.5;
+#define DEBUG_X			(SCREEN_LEFT + 10)
+#define DEBUG_Y			(CENTER_Y-100)
 
-const float DEBUG_X			= SCREEN_LEFT + 10;
-const float DEBUG_Y			= CENTER_Y-100;
+#define SHORTCUTS_X		(CENTER_X - 150)
+#define SHORTCUTS_Y		(CENTER_Y)
 
-const float SHORTCUTS_X		= CENTER_X - 150;
-const float SHORTCUTS_Y		= CENTER_Y;
+#define HELP_X			(SCREEN_LEFT)
+#define HELP_Y			(CENTER_Y)
 
-const float HELP_X			= SCREEN_LEFT;
-const float HELP_Y			= CENTER_Y;
+#define HELP_TEXT_X		(SCREEN_LEFT + 4)
+#define HELP_TEXT_Y		(40)
 
-const float HELP_TEXT_X		= SCREEN_LEFT + 4;
-const float HELP_TEXT_Y		= 40;
+#define INFO_X			(SCREEN_RIGHT)
+#define INFO_Y			(CENTER_Y)
 
-const float INFO_X			= SCREEN_RIGHT;
-const float INFO_Y			= CENTER_Y;
+#define INFO_TEXT_X		(SCREEN_RIGHT - 114)
+#define INFO_TEXT_Y		(40)
 
-const float INFO_TEXT_X		= SCREEN_RIGHT - 114;
-const float INFO_TEXT_Y		= 40;
+#define MENU_WIDTH		(110)
+#define EDIT_X			(CENTER_X)
+#define EDIT_GRAY_Y_STANDARD	(SCREEN_TOP+60)
+#define EDIT_GRAY_Y_REVERSE		(SCREEN_BOTTOM-60)
 
-const float MENU_WIDTH		=	110;
-const float EDIT_X			=	CENTER_X;
-const float EDIT_GRAY_Y		=	GRAY_ARROW_Y;
+#define PLAYER_X			(CENTER_X)
+#define PLAYER_Y			(CENTER_Y)
+#define PLAYER_HEIGHT		(360)
+#define PLAYER_Y_STANDARD	(PLAYER_Y-PLAYER_HEIGHT/2)
+#define PLAYER_Y_REVERSE	(PLAYER_Y+PLAYER_HEIGHT/2)
 
-const float PLAYER_X		=	CENTER_X;
-const float PLAYER_Y		=	SCREEN_TOP;
+#define ACTION_MENU_ITEM_X			(CENTER_X-200)
+#define ACTION_MENU_ITEM_START_Y	(SCREEN_TOP + 24)
+#define ACTION_MENU_ITEM_SPACING_Y	(18)
 
-const float ACTION_MENU_ITEM_X			=	CENTER_X-200;
-const float ACTION_MENU_ITEM_START_Y	=	SCREEN_TOP + 24;
-const float ACTION_MENU_ITEM_SPACING_Y	=	18;
-
-const float NAMING_MENU_ITEM_X			=	CENTER_X-200;
-const float NAMING_MENU_ITEM_START_Y	=	SCREEN_TOP + 24;
-const float NAMING_MENU_ITEM_SPACING_Y	=	18;
+#define NAMING_MENU_ITEM_X			(CENTER_X-200)
+#define NAMING_MENU_ITEM_START_Y	(SCREEN_TOP + 24)
+#define NAMING_MENU_ITEM_SPACING_Y	(18)
 
 CachedThemeMetricF	 TICK_EARLY_SECONDS		("ScreenGameplay","TickEarlySeconds");
 
@@ -248,26 +249,28 @@ ScreenEdit::ScreenEdit() : Screen("ScreenEdit")
 	m_BGAnimation.LoadFromAniDir( THEME->GetPathToB("ScreenEdit background") );
 
 	shiftAnchor = -1;
-	m_SnapDisplay.SetXY( EDIT_X, EDIT_GRAY_Y );
+
+
+	m_SnapDisplay.SetXY( EDIT_X, PLAYER_Y_STANDARD );
 	m_SnapDisplay.Load( PLAYER_1 );
 	m_SnapDisplay.SetZoom( 0.5f );
 
-	m_GrayArrowRowEdit.SetXY( EDIT_X, EDIT_GRAY_Y );
+	m_GrayArrowRowEdit.SetXY( EDIT_X, PLAYER_Y_STANDARD );
 	m_GrayArrowRowEdit.Load( PLAYER_1 );
 	m_GrayArrowRowEdit.SetZoom( 0.5f );
 
-	m_NoteFieldEdit.SetXY( EDIT_X, EDIT_GRAY_Y );
+	m_NoteFieldEdit.SetXY( EDIT_X, PLAYER_Y );
 	m_NoteFieldEdit.SetZoom( 0.5f );
-	m_NoteFieldEdit.Load( &noteData, PLAYER_1, -240, 800, 800 );
+	m_NoteFieldEdit.Load( &noteData, PLAYER_1, -240, 800, PLAYER_HEIGHT*2 );
 
 	m_rectRecordBack.StretchTo( RectI(SCREEN_LEFT, SCREEN_TOP, SCREEN_RIGHT, SCREEN_BOTTOM) );
 	m_rectRecordBack.SetDiffuse( RageColor(0,0,0,0) );
 
-	m_GrayArrowRowRecord.SetXY( EDIT_X, EDIT_GRAY_Y );
+	m_GrayArrowRowRecord.SetXY( EDIT_X, PLAYER_Y );
 	m_GrayArrowRowRecord.Load( PLAYER_1 );
 	m_GrayArrowRowRecord.SetZoom( 1.0f );
 
-	m_NoteFieldRecord.SetXY( EDIT_X, EDIT_GRAY_Y );
+	m_NoteFieldRecord.SetXY( EDIT_X, PLAYER_Y );
 	m_NoteFieldRecord.SetZoom( 1.0f );
 	m_NoteFieldRecord.Load( &noteData, PLAYER_1, -150, 350, 350 );
 
diff --git a/stepmania/src/Song.cpp b/stepmania/src/Song.cpp
index 136a58853c..452cbbfa42 100644
--- a/stepmania/src/Song.cpp
+++ b/stepmania/src/Song.cpp
@@ -900,27 +900,31 @@ void Song::TranslateTitles()
 
 void Song::ReCalculateRadarValuesAndLastBeat()
 {
-	//
-	// calculate radar values and first/last beat
-	//
 	for( unsigned i=0; i<m_apNotes.size(); i++ )
 	{
 		Steps* pNotes = m_apNotes[i];
+
+		/* If it's autogen, radar vals and first/last beat will come from the parent. */
 		if( pNotes->IsAutogen() )
 			continue;
 
+
+		//
+		// calculate radar values
+		//
 		NoteData tempNoteData;
 		pNotes->GetNoteData( &tempNoteData );
 
 		for( int r=0; r<NUM_RADAR_CATEGORIES; r++ )
 		{
-			/* If it's autogen, radar vals come from the parent. */
-			if(pNotes->IsAutogen())
-				continue;
-
-			pNotes->SetRadarValue(r, NoteDataUtil::GetRadarValue( tempNoteData, (RadarCategory)r, m_fMusicLengthSeconds ));
+			float fVal = NoteDataUtil::GetRadarValue( tempNoteData, (RadarCategory)r, m_fMusicLengthSeconds );
+			pNotes->SetRadarValue(r, fVal);
 		}
 
+
+		//
+		// calculate first/last beat
+		//
 		/* Many songs have stray, empty song patterns.  Ignore them, so
 		 * they don't force the first beat of the whole song to 0.  XXX Should
 		 * we just delete them, now that new patterns can be created quickly
diff --git a/stepmania/src/Steps.cpp b/stepmania/src/Steps.cpp
index abd489a694..ce2ec8523f 100644
--- a/stepmania/src/Steps.cpp
+++ b/stepmania/src/Steps.cpp
@@ -173,7 +173,7 @@ void Steps::Decompress() const
 		else
 		{
 			notes->LoadTransformedSlidingWindow( &pdata, notes->GetNumTracks() );
-			NoteDataUtil::FixImpossibleRows( pdata, m_StepsType );
+			NoteDataUtil::FixImpossibleRows( *notes, m_StepsType );
 		}
 	}
 	else if(!notes_comp)