please clang and explicitly override

diff --git a/include/simgrid/s4u/Activity.hpp b/include/simgrid/s4u/Activity.hpp
index 6e894f6..22fa0a7 100644 (file)
--- a/include/simgrid/s4u/Activity.hpp
+++ b/include/simgrid/s4u/Activity.hpp
@@ -33,7 +33,7 @@ protected:
   Activity()  = default;
   virtual ~Activity() = default;
 
-  virtual bool is_assigned() = 0;
+  virtual bool is_assigned() const = 0;
 
   void release_dependencies()
   {

diff --git a/include/simgrid/s4u/Comm.hpp b/include/simgrid/s4u/Comm.hpp
index b202540..0f27f0a 100644 (file)
--- a/include/simgrid/s4u/Comm.hpp
+++ b/include/simgrid/s4u/Comm.hpp
@@ -133,7 +133,7 @@ public:
 
   Actor* get_sender() const;
 
-  bool is_assigned() { return (to_ != nullptr && from_ != nullptr) || (mailbox_ != nullptr); }
+  bool is_assigned() const override { return (to_ != nullptr && from_ != nullptr) || (mailbox_ != nullptr); }
 };
 } // namespace s4u
 } // namespace simgrid

diff --git a/include/simgrid/s4u/Exec.hpp b/include/simgrid/s4u/Exec.hpp
index f9a4007..6cb7870 100644 (file)
--- a/include/simgrid/s4u/Exec.hpp
+++ b/include/simgrid/s4u/Exec.hpp
@@ -81,7 +81,7 @@ public:
   double get_finish_time() const;
   double get_cost() const;
   bool is_parallel() const { return parallel_; }
-  bool is_assigned() { return not hosts_.empty(); }
+  bool is_assigned() const override { return not hosts_.empty(); }
 };
 
 } // namespace s4u

diff --git a/include/simgrid/s4u/Io.hpp b/include/simgrid/s4u/Io.hpp
index 0191c7a..832631d 100644 (file)
--- a/include/simgrid/s4u/Io.hpp
+++ b/include/simgrid/s4u/Io.hpp
@@ -52,7 +52,7 @@ public:
   IoPtr set_size(sg_size_t size);
   IoPtr set_op_type(OpType type);
 
-  bool is_assigned() { return disk_ != nullptr; }
+  bool is_assigned() const override { return disk_ != nullptr; }
 };
 
 } // namespace s4u