comparison base/Event.h @ 1647:29a20719796e single-point

Rework NoteModel commands (not entirely successfully); remove FlexiNoteModel as it has always been almost entirely identical to NoteModel (unlike its layer counterpart)

 #include <stdexcept>
 
 #include <QString>
 
 /**
- * An immutable type used for point and event representation in sparse
- * models, as well as for interchange within the clipboard. An event
- * always has a frame and (possibly empty) label, and optionally has
- * numerical value, level, duration in frames, and a mapped reference
- * frame. Event has an operator< defining a total ordering, by frame
- * first and then by the other properties.
+ * An immutable(-ish) type used for point and event representation in
+ * sparse models, as well as for interchange within the clipboard. An
+ * event always has a frame and (possibly empty) label, and optionally
+ * has numerical value, level, duration in frames, and a mapped
+ * reference frame. Event has an operator< defining a total ordering,
+ * by frame first and then by the other properties.
  * 
  * Event is based on the Clipboard::Point type up to SV v3.2.1 and is
  * intended also to replace the custom point types previously found in
  * sparse models.
  */

         m_duration(duration), m_referenceFrame(0), m_label(label) {
         if (m_duration < 0) throw std::logic_error("duration must be >= 0");
     }
 
     Event(const Event &event) =default;
+
+    // We would ideally like Event to be immutable - but we have to
+    // have these because otherwise we can't put Events in vectors
+    // etc. Let's call it conceptually immutable
     Event &operator=(const Event &event) =default;
     Event &operator=(Event &&event) =default;
     
     sv_frame_t getFrame() const { return m_frame; }