From 9f475d542771b8435e84e764f9f1ea4a2159fc2b Mon Sep 17 00:00:00 2001
From: Paul Davis <paul@linuxaudiosystems.com>
Date: Sun, 24 Sep 2023 18:41:51 -0600
Subject: [PATCH] avoid double call to ::model_changed() during set up of a
 MidiRegionView

There is a DisplaySuspender in MidiRegionView::init(), and another invoked
during the addition of a RegionView to a StreamView. Before this change, both
suspenders were created to invoke a full "model-changed" redisplay. Since every
RegionView must be added to a StreamView to be visible, we only need the second
suspender to invoke the model-change. The first one can be just a view
change (and probably isn't necessary at all, but I did not want to change that
much at this point in time).
---
 gtk2_ardour/midi_region_view.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc
index 7e1106c6d6..838d3faf63 100644
--- a/gtk2_ardour/midi_region_view.cc
+++ b/gtk2_ardour/midi_region_view.cc
@@ -256,7 +256,7 @@ MidiRegionView::MidiRegionView (const MidiRegionView& other, std::shared_ptr<Mid
 void
 MidiRegionView::init (bool /*wfd*/)
 {
-	DisplaySuspender ds (*this);
+	DisplaySuspender ds (*this, true);
 
 	PublicEditor::DropDownKeys.connect (sigc::mem_fun (*this, &MidiRegionView::drop_down_keys));