From 6ac9c52b5f2511cde8a30816a9a9626f37193da7 Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Sat, 2 Apr 2022 09:12:57 -0600 Subject: [PATCH] remove debug output and some mistakenly commented lines --- gtk2_ardour/midi_region_view.cc | 30 ++++++------------------------ 1 file changed, 6 insertions(+), 24 deletions(-) diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc index 7932e5946d..3cba8036c1 100644 --- a/gtk2_ardour/midi_region_view.cc +++ b/gtk2_ardour/midi_region_view.cc @@ -902,8 +902,6 @@ MidiRegionView::display_model (boost::shared_ptr model) _model->ContentsChanged.connect (content_connection, invalidator (*this), boost::bind (&MidiRegionView::model_changed, this), gui_context()); /* Don't signal as nobody else needs to know until selection has been altered. */ clear_events(); - // PBD::stacktrace (std::cerr, 8); - model_changed (); } @@ -1095,8 +1093,6 @@ MidiRegionView::model_changed() return; } - // PBD::stacktrace (std::cerr, 8); - Timing t; if (_active_notes) { @@ -1121,7 +1117,7 @@ MidiRegionView::model_changed() } for (_optimization_iterator = _events.begin(); _optimization_iterator != _events.end(); ++_optimization_iterator) { - // _optimization_iterator->second->invalidate(); + _optimization_iterator->second->invalidate(); } bool empty_when_starting = _events.empty(); @@ -1135,7 +1131,6 @@ MidiRegionView::model_changed() NoteBase* cne; - if (empty_when_starting) { for (MidiModel::Notes::iterator n = notes.begin(); n != notes.end(); ++n) { boost::shared_ptr note (*n); @@ -1154,7 +1149,6 @@ MidiRegionView::model_changed() } } } - } if (!empty_when_starting) { MidiModel::Notes::iterator f; @@ -1196,8 +1190,6 @@ MidiRegionView::model_changed() } } - std::cerr << this << " Adding " << missing_notes.size() << " missing notes\n"; - for (MidiModel::Notes::iterator n = missing_notes.begin(); n != missing_notes.end(); ++n) { boost::shared_ptr note (*n); NoteBase* cne; @@ -1235,7 +1227,7 @@ MidiRegionView::model_changed() _pending_note_selection.clear (); t.update (); - std::cerr << "REDISPLAY of " << region()->name() << " complete after " << t.elapsed_msecs() << std::endl; + std::cerr << "REDISPLAY(model) of " << region()->name() << " complete after " << t.elapsed_msecs() << std::endl; } void @@ -1245,8 +1237,6 @@ MidiRegionView::view_changed() return; } - // PBD::stacktrace (std::cerr, 8); - Timing t; if (_active_notes) { @@ -1277,20 +1267,12 @@ MidiRegionView::view_changed() NoteBase* cne = i->second; - bool visible = cne->item()->visible(); - - if ((sus = dynamic_cast(cne))) { - - if (visible) { + if (cne->item()->visible()) { + if ((sus = dynamic_cast(cne))) { update_sustained (sus); } - } else if ((hit = dynamic_cast(cne))) { - - if (visible) { - update_hit (hit); - } - + update_hit (hit); } ++i; @@ -1307,7 +1289,7 @@ MidiRegionView::view_changed() update_patch_changes (); t.update (); - std::cerr << "REDISPLAY of " << region()->name() << " complete after " << t.elapsed_msecs() << std::endl; + std::cerr << "REDISPLAY(view) of " << region()->name() << " complete after " << t.elapsed_msecs() << std::endl; } void