Use remove() instead of close() to solve problem of midi clip playing…

… after deletion
LMMS · Sep 23, 2024 · e1269e7 · e1269e7
1 parent 858330a
commit e1269e7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/gui/clips/AutomationClipView.cpp b/src/gui/clips/AutomationClipView.cpp
@@ -534,8 +534,9 @@ bool AutomationClipView::splitClip(const TimePos pos)
 	rightClip->movePosition(splitPos);
 	rightClip->changeLength(m_initialClipEnd - splitPos);
 
+	remove();
+
 	m_clip->getTrack()->restoreJournallingState();
-	close();
 	return true;
 }
 

diff --git a/src/gui/clips/MidiClipView.cpp b/src/gui/clips/MidiClipView.cpp
@@ -726,8 +726,9 @@ bool MidiClipView::splitClip(const TimePos pos)
 	rightClip->movePosition(splitPos);
 	rightClip->updateLength();
 
+	remove();
+
 	m_clip->getTrack()->restoreJournallingState();
-	close();
 	return true;
 }