changeset 1795:94b488d4b299

The copy & move operators are not actually being used, so probably safer to delete them unless we know otherwise - this is not all that simple a class
author Chris Cannam
date Mon, 30 Sep 2019 12:36:44 +0100
parents 71e97de5053f
children ff8c57c364a0
files base/EventSeries.h
diffstat 1 files changed, 5 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/base/EventSeries.h	Wed Sep 25 13:47:46 2019 +0100
+++ b/base/EventSeries.h	Mon Sep 30 12:36:44 2019 +0100
@@ -39,6 +39,8 @@
  * order of start frame. Inserting (or deleting) events in the middle
  * does work, and should be acceptable in interactive use, but it is
  * very slow in bulk.
+ *
+ * EventSeries is not thread-safe.
  */
 class EventSeries : public XmlExportable
 {
@@ -47,8 +49,9 @@
     ~EventSeries() =default;
 
     EventSeries(const EventSeries &) =default;
-    EventSeries &operator=(const EventSeries &) =default;
-    EventSeries &operator=(EventSeries &&) =default;
+
+    EventSeries &operator=(const EventSeries &) =delete;
+    EventSeries &operator=(EventSeries &&) =delete;
     
     bool operator==(const EventSeries &other) const {
         return m_events == other.m_events;