smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-06 06:44:57 +01:00
Code Issues Projects Releases Wiki Activity

avoid double call to ::model_changed() during set up of a MidiRegionView

Browse source 
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).
This commit is contained in:
Paul Davis 2023-09-24 18:41:51 -06:00
parent 1f13b311fd
commit 9f475d5427
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
gtk2_ardour/midi_region_view.cc
View file

@ -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));