From: Millian Poquet <millian.poquet@inria.fr>
Date: Mon, 18 Jan 2016 17:46:52 +0000 (+0100)
Subject: Warning fix: addition of the override specifier for all methods that needed it
X-Git-Tag: v3_13~1210^2^2~4
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/f7e964cd446a66ef80776a5f1e10e401f822e20c

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 be3b5617b3..56156334ec 100644
--- 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 b116ed6b5b..f3b7c861bc 100644
--- 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 545cb1b571..d81510bc2b 100644
--- 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 1ed2fa0300..6b0d8b2376 100644
--- 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 29640c0d26..f97dfcc796 100644
--- 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 1122d7e3d8..936753476c 100644
--- 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\