From 2784ce99e192777f2c38a4b37193aaa048cdb9ff Mon Sep 17 00:00:00 2001 From: Paul Davis Date: Thu, 6 Mar 2014 13:27:26 -0500 Subject: [PATCH] comment edit --- gtk2_ardour/note_base.h | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/gtk2_ardour/note_base.h b/gtk2_ardour/note_base.h index 8ee3a4d277..6528b28659 100644 --- a/gtk2_ardour/note_base.h +++ b/gtk2_ardour/note_base.h @@ -42,26 +42,21 @@ 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: 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 *); -- 2.30.2