followup to the previous tempo-dragging commit: use argument to Drag::aborted() to...
[ardour.git] / gtk2_ardour / ghostregion.cc
index 4c2f17d29d0c0c659da9b86d741b6c59ee887d3f..31e6de390ba23186068c8510da2bcfd80e249853 100644 (file)
@@ -20,7 +20,6 @@
 #include "evoral/Note.hpp"
 #include "ardour_ui.h"
 #include "automation_time_axis.h"
-#include "canvas-hit.h"
 #include "canvas-note.h"
 #include "ghostregion.h"
 #include "midi_streamview.h"
@@ -105,7 +104,7 @@ GhostRegion::is_automation_ghost()
 AudioGhostRegion::AudioGhostRegion(TimeAxisView& tv, TimeAxisView& source_tv, double initial_unit_pos)
        : GhostRegion(tv.ghost_group(), tv, source_tv, initial_unit_pos)
 {
-       
+
 }
 
 void
@@ -163,6 +162,7 @@ AudioGhostRegion::set_colors ()
  */
 MidiGhostRegion::MidiGhostRegion(TimeAxisView& tv, TimeAxisView& source_tv, double initial_unit_pos)
        : GhostRegion(tv.ghost_group(), tv, source_tv, initial_unit_pos)
+       , _optimization_iterator (events.end ())
 {
        base_rect->lower_to_bottom();
        update_range ();
@@ -176,6 +176,7 @@ MidiGhostRegion::MidiGhostRegion(TimeAxisView& tv, TimeAxisView& source_tv, doub
  */
 MidiGhostRegion::MidiGhostRegion(MidiStreamView& msv, TimeAxisView& source_tv, double initial_unit_pos)
        : GhostRegion(msv.midi_underlay_group, msv.trackview(), source_tv, initial_unit_pos)
+       , _optimization_iterator (events.end ())
 {
        base_rect->lower_to_bottom();
        update_range ();
@@ -185,33 +186,19 @@ MidiGhostRegion::MidiGhostRegion(MidiStreamView& msv, TimeAxisView& source_tv, d
 
 MidiGhostRegion::~MidiGhostRegion()
 {
-       //clear_events();
-}
-
-MidiGhostRegion::Event::Event(ArdourCanvas::CanvasNoteEvent* e)
-       : event(e)
-{
-}
-
-MidiGhostRegion::Note::Note(ArdourCanvas::CanvasNote* n, ArdourCanvas::Group* g)
-       : Event(n)
-{
-       rect = new ArdourCanvas::SimpleRect(*g, n->x1(), n->y1(), n->x2(), n->y2());
-}
-
-MidiGhostRegion::Note::~Note()
-{
-       //delete rect;
+       clear_events ();
 }
 
-MidiGhostRegion::Hit::Hit(ArdourCanvas::CanvasHit* h, ArdourCanvas::Group*)
-       : Event(h)
+MidiGhostRegion::Event::Event (ArdourCanvas::CanvasNoteEvent* e, ArdourCanvas::Group* g)
+       : event (e)
 {
-       cerr << "Hit ghost item does not work yet" << endl;
+       rect = new ArdourCanvas::SimpleRect (*g, e->x1(), e->y1(), e->x2(), e->y2());
 }
 
-MidiGhostRegion::Hit::~Hit()
+MidiGhostRegion::Event::~Event ()
 {
+       /* event is not ours to delete */
+       delete rect;
 }
 
 void
@@ -241,16 +228,13 @@ MidiGhostRegion::set_height ()
 void
 MidiGhostRegion::set_colors()
 {
-       MidiGhostRegion::Note* note;
        guint fill = source_track_color(200);
 
        GhostRegion::set_colors();
 
        for (EventList::iterator it = events.begin(); it != events.end(); ++it) {
-               if ((note = dynamic_cast<MidiGhostRegion::Note*>(*it)) != 0) {
-                       note->rect->property_fill_color_rgba() = fill;
-                       note->rect->property_outline_color_rgba() =  ARDOUR_UI::config()->canvasvar_GhostTrackMidiOutline.get();
-               }
+               (*it)->rect->property_fill_color_rgba() = fill;
+               (*it)->rect->property_outline_color_rgba() = ARDOUR_UI::config()->canvasvar_GhostTrackMidiOutline.get();
        }
 }
 
@@ -263,21 +247,18 @@ MidiGhostRegion::update_range ()
                return;
        }
 
-       MidiGhostRegion::Note* note;
        double const h = trackview.current_height() / double (mv->contents_note_range ());
 
        for (EventList::iterator it = events.begin(); it != events.end(); ++it) {
-               if ((note = dynamic_cast<MidiGhostRegion::Note*>(*it)) != 0) {
-                       uint8_t const note_num = note->event->note()->note();
-
-                       if (note_num < mv->lowest_note() || note_num > mv->highest_note()) {
-                               note->rect->hide();
-                       } else {
-                               note->rect->show();
-                               double const y = trackview.current_height() - (note_num + 1 - mv->lowest_note()) * h + 1;
-                               note->rect->property_y1() = y;
-                               note->rect->property_y2() = y + h;
-                       }
+               uint8_t const note_num = (*it)->event->note()->note();
+
+               if (note_num < mv->lowest_note() || note_num > mv->highest_note()) {
+                       (*it)->rect->hide();
+               } else {
+                       (*it)->rect->show();
+                       double const y = trackview.current_height() - (note_num + 1 - mv->lowest_note()) * h + 1;
+                       (*it)->rect->property_y1() = y;
+                       (*it)->rect->property_y2() = y + h;
                }
        }
 }
@@ -285,11 +266,11 @@ MidiGhostRegion::update_range ()
 void
 MidiGhostRegion::add_note(ArdourCanvas::CanvasNote* n)
 {
-       Note* note = new Note(n, group);
-       events.push_back(note);
+       Event* event = new Event (n, group);
+       events.push_back (event);
 
-       note->rect->property_fill_color_rgba() = source_track_color(200);
-       note->rect->property_outline_color_rgba() = ARDOUR_UI::config()->canvasvar_GhostTrackMidiOutline.get();
+       event->rect->property_fill_color_rgba() = source_track_color(200);
+       event->rect->property_outline_color_rgba() = ARDOUR_UI::config()->canvasvar_GhostTrackMidiOutline.get();
 
        MidiStreamView* mv = midi_view();
 
@@ -297,21 +278,15 @@ MidiGhostRegion::add_note(ArdourCanvas::CanvasNote* n)
                const uint8_t note_num = n->note()->note();
 
                if (note_num < mv->lowest_note() || note_num > mv->highest_note()) {
-                       note->rect->hide();
+                       event->rect->hide();
                } else {
                        const double y = mv->note_to_y(note_num);
-                       note->rect->property_y1() = y;
-                       note->rect->property_y2() = y + mv->note_height();
+                       event->rect->property_y1() = y;
+                       event->rect->property_y2() = y + mv->note_height();
                }
        }
 }
 
-void
-MidiGhostRegion::add_hit(ArdourCanvas::CanvasHit* /*h*/)
-{
-       //events.push_back(new Hit(h, group));
-}
-
 void
 MidiGhostRegion::clear_events()
 {
@@ -320,5 +295,64 @@ MidiGhostRegion::clear_events()
        }
 
        events.clear();
+       _optimization_iterator = events.end ();
 }
 
+/** Update the x positions of our representation of a parent's note.
+ *  @param parent The CanvasNote from the parent MidiRegionView.
+ */
+void
+MidiGhostRegion::update_note (ArdourCanvas::CanvasNote* parent)
+{
+       Event* ev = find_event (parent);
+       if (!ev) {
+               return;
+       }
+
+       double const x1 = parent->property_x1 ();
+       double const x2 = parent->property_x2 ();
+       ev->rect->property_x1 () = x1;
+       ev->rect->property_x2 () = x2;
+}
+
+void
+MidiGhostRegion::remove_note (ArdourCanvas::CanvasNoteEvent* note)
+{
+       Event* ev = find_event (note);
+       if (!ev) {
+               return;
+       }
+
+       events.remove (ev);
+       delete ev;
+       _optimization_iterator = events.end ();
+}
+
+/** Given a note in our parent region (ie the actual MidiRegionView), find our
+ *  representation of it.
+ *  @return Our Event, or 0 if not found.
+ */
+
+MidiGhostRegion::Event *
+MidiGhostRegion::find_event (ArdourCanvas::CanvasNoteEvent* parent)
+{
+       /* we are using _optimization_iterator to speed up the common case where a caller
+          is going through our notes in order.
+       */
+
+       if (_optimization_iterator != events.end()) {
+               ++_optimization_iterator;
+       }
+
+       if (_optimization_iterator != events.end() && (*_optimization_iterator)->event == parent) {
+               return *_optimization_iterator;
+       }
+
+       for (_optimization_iterator = events.begin(); _optimization_iterator != events.end(); ++_optimization_iterator) {
+               if ((*_optimization_iterator)->event == parent) {
+                       return *_optimization_iterator;
+               }
+       }
+
+       return 0;
+}