DlgTrackInfo: prevent wipe when quickly applying twice

Fixes #12963

Co-authored-by: Uwe Klotz <uwe.klotz@gmail.com>

src/library/dlgtrackinfo.cpp

@@ -621,8 +621,13 @@ void DlgTrackInfo::saveTrack() {
     }
 
     // Update the cached track
-    // The dialog is updated and repopulated by the Track::changed() signal.
-    m_pLoadedTrack->replaceRecord(std::move(m_trackRecord), std::move(m_pBeatsClone));
+    // Update the cached track
+    //
+    // If replaceRecord() returns true then both m_trackRecord and m_pBeatsClone
+    // will be updated by the subsequent Track::changed() signal to keep them
+    // synchronized with the track. Otherwise the track has not been modified and
+    // both members must remain valid. Do not use std::move() for passing arguments!
+    m_pLoadedTrack->replaceRecord(m_trackRecord, m_pBeatsClone);
 }
 
 void DlgTrackInfo::clear() {