From 858330ab80d5e7908aca1adc86451d107a9382a9 Mon Sep 17 00:00:00 2001
From: regulus79 <117475203+regulus79@users.noreply.github.com>
Date: Sun, 22 Sep 2024 16:23:10 -0400
Subject: [PATCH] Initialize variables in AutomationClip's copy constructor

---
 src/core/AutomationClip.cpp          | 5 ++++-
 src/gui/clips/AutomationClipView.cpp | 4 ----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/core/AutomationClip.cpp b/src/core/AutomationClip.cpp
index dc496fa044a..7689324dfbc 100644
--- a/src/core/AutomationClip.cpp
+++ b/src/core/AutomationClip.cpp
@@ -87,7 +87,10 @@ AutomationClip::AutomationClip( const AutomationClip & _clip_to_copy ) :
 	m_autoTrack( _clip_to_copy.m_autoTrack ),
 	m_objects( _clip_to_copy.m_objects ),
 	m_tension( _clip_to_copy.m_tension ),
-	m_progressionType( _clip_to_copy.m_progressionType )
+	m_progressionType(_clip_to_copy.m_progressionType),
+	m_dragging(false),
+	m_isRecording(_clip_to_copy.m_isRecording),
+	m_lastRecordedValue(0)
 {
 	// Locks the mutex of the copied AutomationClip to make sure it
 	// doesn't change while it's being copied
diff --git a/src/gui/clips/AutomationClipView.cpp b/src/gui/clips/AutomationClipView.cpp
index 0ebd3e24a2c..e20c3161f60 100644
--- a/src/gui/clips/AutomationClipView.cpp
+++ b/src/gui/clips/AutomationClipView.cpp
@@ -533,10 +533,6 @@ bool AutomationClipView::splitClip(const TimePos pos)
 
 	rightClip->movePosition(splitPos);
 	rightClip->changeLength(m_initialClipEnd - splitPos);
-
-	// For some reason, the new clips sometime randomly put themselves in record mode. This is a temportary fix which forces them to match the original clip.
-	rightClip->setRecording(m_clip->isRecording());
-	leftClip->setRecording(m_clip->isRecording());
 	
 	m_clip->getTrack()->restoreJournallingState();
 	close();