Assignment operators should return non-"const" references (sonar).

[simgrid.git] / src / mc / api / strategy / Strategy.hpp

diff --git a/src/mc/api/strategy/Strategy.hpp b/src/mc/api/strategy/Strategy.hpp
index ef5b87f..a530789 100644 (file)
--- a/src/mc/api/strategy/Strategy.hpp
+++ b/src/mc/api/strategy/Strategy.hpp
@@ -17,10 +17,12 @@ protected:
   std::map<aid_t, ActorState> actors_to_run_;
 
 public:
-  virtual ~Strategy()                                      = default;
-  void operator=(const Strategy&)
+  Strategy()                = default;
+  virtual ~Strategy()       = default;
+  Strategy(const Strategy&) = delete;
+  Strategy& operator=(const Strategy&)
   { /* nothing to copy over while cloning */
-    return;
+    return *this;
   }
 
   virtual std::pair<aid_t, int> next_transition() const = 0;