From 6e6963d5c78a73b8ccab632fd61135d81a1876c7 Mon Sep 17 00:00:00 2001
From: Glenn Maynard <gmaynard@users.sourceforge.net>
Date: Tue, 17 Jun 2003 20:28:32 +0000
Subject: [PATCH] try to fix snapping with hold notes before, we were setting
 spurious TAP_HOLD_HEADs, and we weren't checking hold tails, which would
 cause taps to be moved inside the tail end of holds

---
 stepmania/src/NoteData.cpp | 22 +++++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/stepmania/src/NoteData.cpp b/stepmania/src/NoteData.cpp
index d2c5a1db12..1af1dc151d 100644
--- a/stepmania/src/NoteData.cpp
+++ b/stepmania/src/NoteData.cpp
@@ -1160,7 +1160,7 @@ void NoteDataUtil::SnapToNearestNoteType( NoteData &in, NoteType nt1, NoteType n
 	int iNoteIndexBegin = BeatToNoteRow( fBeginBeat );
 	int iNoteIndexEnd = BeatToNoteRow( fEndBeat );
 
-	//ConvertHoldNotesTo2sAnd3s();
+	in.ConvertHoldNotesTo2sAnd3s();
 
 	// iterate over all TapNotes in the interval and snap them
 	for( int i=iNoteIndexBegin; i<=iNoteIndexEnd; i++ )
@@ -1180,19 +1180,31 @@ void NoteDataUtil::SnapToNearestNoteType( NoteData &in, NoteType nt1, NoteType n
 
 		for( int c=0; c<in.GetNumTracks(); c++ )
 		{
+			if( iOldIndex == iNewIndex )
+				continue;
+
 			TapNote note = in.GetTapNote(c, iOldIndex);
 			if( note == TAP_EMPTY )
 				continue;
 
 			in.SetTapNote(c, iOldIndex, TAP_EMPTY);
-			// HoldNotes override TapNotes
-			if(in.GetTapNote(c, iNewIndex) == TAP_TAP)
-				note = TAP_HOLD_HEAD;
+
+			const TapNote oldnote = in.GetTapNote(c, iNewIndex);
+			if( note == TAP_TAP &&
+				(oldnote == TAP_HOLD_HEAD || oldnote == TAP_HOLD_TAIL) )
+				continue; // HoldNotes override TapNotes
+
+			/* If two hold note boundaries are getting snapped together,
+			 * merge them. */
+			if( (note == TAP_HOLD_HEAD && oldnote == TAP_HOLD_TAIL) ||
+				(note == TAP_HOLD_TAIL && oldnote == TAP_HOLD_HEAD))
+				note = TAP_EMPTY;
+			
 			in.SetTapNote(c, iNewIndex, note );
 		}
 	}
 
-	//Convert2sAnd3sToHoldNotes();
+	in.Convert2sAnd3sToHoldNotes();
 }