X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Fnote_base.h;h=5e52be0732e7577288e16838163147cde433e258;hb=0ea1238479c9692a2b212cc226b783dce4f1e3c6;hp=8ee3a4d27790496a15f16ddfd5772b1fd19d47ad;hpb=e466ce40ad1ba591543020cb7c0aa15dbebef81e;p=ardour.git diff --git a/gtk2_ardour/note_base.h b/gtk2_ardour/note_base.h index 8ee3a4d277..5e52be0732 100644 --- a/gtk2_ardour/note_base.h +++ b/gtk2_ardour/note_base.h @@ -42,29 +42,25 @@ namespace ArdourCanvas { /** This manages all the event handling for any MIDI event on the canvas. * - * This is not actually a canvas item itself to avoid the dreaded diamond, - * since various types of canvas items (Note (rect), Hit (diamond), etc) - * need to share this functionality but can't share an ancestor. + * This is not actually a canvas item itself to avoid the dreaded diamond + * inheritance pattern, since various types of canvas items (Note (rect), Hit + * (diamond), etc) need to share this functionality but can't share an + * ancestor. * * Note: Because of this, derived classes need to manually bounce events to * on_event, it won't happen automatically. - * - * A newer, better canvas should remove the need for all the ugly here. */ + class NoteBase : public sigc::trackable { -public: + public: typedef Evoral::Note NoteType; - NoteBase ( - MidiRegionView& region, - bool, - const boost::shared_ptr note = boost::shared_ptr() - ); - + NoteBase (MidiRegionView& region, bool, const boost::shared_ptr note = boost::shared_ptr()); virtual ~NoteBase (); void set_item (ArdourCanvas::Item *); + ArdourCanvas::Item* item() const { return _item; } static PBD::Signal1 NoteBaseDeleted; @@ -142,7 +138,7 @@ public: static const uint32_t midi_channel_colors[16]; bool mouse_near_ends () const; - bool big_enough_to_trim () const; + virtual bool big_enough_to_trim () const; protected: enum State { None, Pressed, Dragging };