Warning fix: addition of the override specifier for all methods that needed it

diff --git a/src/surf/cpu_ti.hpp b/src/surf/cpu_ti.hpp
index be3b561..5615633 100644 (file)
--- a/src/surf/cpu_ti.hpp
+++ b/src/surf/cpu_ti.hpp
@@ -156,7 +156,7 @@ public:
                           int pstate, double speedScale,
                           tmgr_trace_t speedTrace, int core,
                           int initiallyOn,
-                          tmgr_trace_t state_trace);
+                          tmgr_trace_t state_trace) override;
   double shareResources(double now) override;
   void updateActionsState(double now, double delta) override;
   void addTraces() override;

diff --git a/src/surf/network_interface.hpp b/src/surf/network_interface.hpp
index b116ed6..f3b7c86 100644 (file)
--- a/src/surf/network_interface.hpp
+++ b/src/surf/network_interface.hpp
@@ -232,7 +232,7 @@ public:
   virtual int sharingPolicy();
 
   /** @brief Check if the Link is used */
-  bool isUsed();
+  bool isUsed() override;
 
   void turnOn() override;
   void turnOff() override;

diff --git a/src/surf/storage_interface.hpp b/src/surf/storage_interface.hpp
index 545cb1b..d81510b 100644 (file)
--- a/src/surf/storage_interface.hpp
+++ b/src/surf/storage_interface.hpp
@@ -138,7 +138,7 @@ public:
   ~Storage();
 
   /** @brief Check if the Storage is used (if an action currently uses its resources) */
-  bool isUsed();
+  bool isUsed() override;
 
   /**
    * @brief Update the state of the current Storage
@@ -147,7 +147,7 @@ public:
    * @param value [description]
    * @param date [description]
    */
-  void updateState(tmgr_trace_event_t event_type, double value, double date);
+  void updateState(tmgr_trace_event_t event_type, double value, double date) override;
 
   void turnOn() override;
   void turnOff() override;

diff --git a/src/surf/vm_hl13.hpp b/src/surf/vm_hl13.hpp
index 1ed2fa0..6b0d8b2 100644 (file)
--- a/src/surf/vm_hl13.hpp
+++ b/src/surf/vm_hl13.hpp
@@ -34,7 +34,7 @@ public:
   ~VMHL13Model(){};
 
   VirtualMachine *createVM(const char *name, sg_host_t host_PM) override;
-  double shareResources(double now);
+  double shareResources(double now) override;
   void adjustWeightOfDummyCpuActions() override {};
   void updateActionsState(double /*now*/, double /*delta*/) override;
 };
@@ -56,8 +56,8 @@ public:
 
   void migrate(sg_host_t ind_dst_pm) override;
 
-  void setBound(double bound);
-  void setAffinity(Cpu *cpu, unsigned long mask);
+  void setBound(double bound) override;
+  void setAffinity(Cpu *cpu, unsigned long mask) override;
 };
 
 /**********

diff --git a/src/xbt/memory_map.cpp b/src/xbt/memory_map.cpp
index 29640c0..f97dfcc 100644 (file)
--- a/src/xbt/memory_map.cpp
+++ b/src/xbt/memory_map.cpp
@@ -167,7 +167,7 @@ XBT_PRIVATE std::vector<VmMap> get_memory_map(pid_t pid)
 
   std::free(line);
   std::fclose(fp);
-  return std::move(ret);
+  return std::move(ret); // MPOQUET FIXME: this line generates a warning on clang 3.7.0
 #else
   /* On FreeBSD, kinfo_getvmmap() could be used but mmap() support is disabled
      anyway. */

diff --git a/src/xbt/snprintf.c b/src/xbt/snprintf.c
index 1122d7e..9367534 100644 (file)
--- a/src/xbt/snprintf.c
+++ b/src/xbt/snprintf.c
@@ -432,6 +432,8 @@ int portable_vsnprintf(char *str, size_t str_m, const char *fmt,
 
 /* declarations */
 
+/* MPOQUET FIXME: the __foo__ function generates a warning on clang 3.7.0
+   MPOQUET FIXME: the credits is unused if __foo__ is commented*/
 static char credits[] = "\n\
   @(#)snprintf.c, v2.2: Mark Martinec, <mark.martinec@ijs.si>\n\
   @(#)snprintf.c, v2.2: Copyright 1999, Mark Martinec. Frontier Artistic License applies.\n\