[sonar] Declare functions "const" in src/instr/.

diff --git a/include/simgrid/jedule/jedule_platform.hpp b/include/simgrid/jedule/jedule_platform.hpp
index 9079e9e..8097776 100644 (file)
--- a/include/simgrid/jedule/jedule_platform.hpp
+++ b/include/simgrid/jedule/jedule_platform.hpp
@@ -30,9 +30,9 @@ public:
 
   const char* get_cname() const { return name.c_str(); }
   void set_parent(Container* parent) { parent_ = parent; }
-  bool has_children() { return not children_.empty(); }
+  bool has_children() const { return not children_.empty(); }
   int get_child_position(const Container* child) const;
-  unsigned int get_id_by_name(const char* name) { return name2id.at(name); }
+  unsigned int get_id_by_name(const char* name) const { return name2id.at(name); }
 
   void add_child(Container* child);
   void add_resources(std::vector<sg_host_t> hosts);

diff --git a/src/instr/instr_paje_events.hpp b/src/instr/instr_paje_events.hpp
index 744ada9..f26a206 100644 (file)
--- a/src/instr/instr_paje_events.hpp
+++ b/src/instr/instr_paje_events.hpp
@@ -81,7 +81,7 @@ public:
   static xbt::signal<void(StateEvent&)> on_destruction;
   StateEvent(Container* container, Type* type, e_event_type event_type, EntityValue* value, TIData* extra);
   ~StateEvent() { on_destruction(*this); }
-  bool has_extra() { return extra_ != nullptr; }
+  bool has_extra() const { return extra_ != nullptr; }
   void print() override;
 };
 

diff --git a/src/instr/instr_paje_types.hpp b/src/instr/instr_paje_types.hpp
index 2ce289b..e23ddc2 100644 (file)
--- a/src/instr/instr_paje_types.hpp
+++ b/src/instr/instr_paje_types.hpp
@@ -34,13 +34,13 @@ public:
        Type* father);
   virtual ~Type() = default;
 
-  long long int get_id() { return id_; }
+  long long int get_id() const { return id_; }
   const std::string& get_name() const { return name_; }
-  const char* get_cname() { return name_.c_str(); }
+  const char* get_cname() const { return name_.c_str(); }
   const std::string& get_color() const { return color_; }
   Type* get_father() const { return father_; }
-  const std::map<std::string, std::unique_ptr<Type>>& get_children() { return children_; }
-  bool is_colored() { return not color_.empty(); }
+  const std::map<std::string, std::unique_ptr<Type>>& get_children() const { return children_; }
+  bool is_colored() const { return not color_.empty(); }
 
   Type* by_name(const std::string& name);
   LinkType* by_name_or_create(const std::string& name, Type* source, Type* dest);

diff --git a/src/instr/instr_smpi.hpp b/src/instr/instr_smpi.hpp
index aac3353..910f702 100644 (file)
--- a/src/instr/instr_smpi.hpp
+++ b/src/instr/instr_smpi.hpp
@@ -33,7 +33,7 @@ public:
   std::string previous_filename;
   int previous_linenumber;
 
-  std::string get_composed_key()
+  std::string get_composed_key() const
   {
     return previous_filename + ':' + std::to_string(previous_linenumber) + ':' + filename + ':' +
            std::to_string(linenumber);