From 9c356ad09b879430dd2928a6ac49932b23b1405f Mon Sep 17 00:00:00 2001
From: Paul Giblock <drfaygo@gmail.com>
Date: Sat, 13 Sep 2008 21:56:07 +0000
Subject: [PATCH] Improved timeline zero-length loop avoidance

git-svn-id: https://lmms.svn.sf.net/svnroot/lmms/trunk/lmms@1604 0778d3d1-df1d-0410-868b-ea421aaaa00d
---
 src/core/timeline.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/core/timeline.cpp b/src/core/timeline.cpp
index 5633ab4f3..742c7e200 100644
--- a/src/core/timeline.cpp
+++ b/src/core/timeline.cpp
@@ -354,12 +354,11 @@ void timeLine::mouseMoveEvent( QMouseEvent * _me )
 			if( m_loopPos[0] == m_loopPos[1] )
 			{
 				// Note, swap 1 and 0 below and the behavior "skips" the other
-				// marking instead of pushing it.  However, I think pushing the
-				// other marker makes more sense it keeps the markers in order
+				// marking instead of pushing it.
 				if( m_action == MoveLoopBegin ) 
-					m_loopPos[1] += midiTime::ticksPerTact();
-				else
 					m_loopPos[0] -= midiTime::ticksPerTact();
+				else
+					m_loopPos[1] += midiTime::ticksPerTact();
 			}
 			update();
 			break;