comment edit

diff --git a/gtk2_ardour/note_base.h b/gtk2_ardour/note_base.h
index 8ee3a4d27790496a15f16ddfd5772b1fd19d47ad..6528b286591dfe51e2a0f9e8e15804e5053c795e 100644 (file)
--- 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<ARDOUR::MidiModel::TimeType> NoteType;
 
-       NoteBase (
-               MidiRegionView&                   region,
-               bool,
-               const boost::shared_ptr<NoteType> note = boost::shared_ptr<NoteType>()
-               );
-
+       NoteBase (MidiRegionView& region, bool, const boost::shared_ptr<NoteType> note = boost::shared_ptr<NoteType>());
        virtual ~NoteBase ();
 
        void set_item (ArdourCanvas::Item *);