From f47e517c86c07580dfeee769f34f1226c9336044 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Wed, 2 Nov 2022 02:05:58 +0100 Subject: [PATCH] Fix duplicate undo crash Double-clicking on a line in internal edit mode adds a new point in the line. In this case LineDrag need not manage automation events because the "add" functions already do. see AutomationTimeAxisView::add_automation_event and AudioRegionView::add_gain_point_event --- gtk2_ardour/editor_drag.cc | 4 ---- 1 file changed, 4 deletions(-) diff --git a/gtk2_ardour/editor_drag.cc b/gtk2_ardour/editor_drag.cc index d66f331db2..f2ab3e2ac3 100644 --- a/gtk2_ardour/editor_drag.cc +++ b/gtk2_ardour/editor_drag.cc @@ -4945,8 +4945,6 @@ LineDrag::finished (GdkEvent* event, bool movement_occurred) AutomationTimeAxisView* atv; - _editor->begin_reversible_command (_("add automation point")); - if ((atv = dynamic_cast(_editor->clicked_axisview)) != 0) { timepos_t where = grab_time (); @@ -4963,8 +4961,6 @@ LineDrag::finished (GdkEvent* event, bool movement_occurred) arv->add_gain_point_event (&arv->get_gain_line()->grab_item(), event, false); } } - - _editor->commit_reversible_command (); } }