Xcos MVC: remove useless 'static' and add 'const' qualifiers 54/15354/5
Paul Bignier [Thu, 9 Oct 2014 09:39:37 +0000 (11:39 +0200)]
 * Functions were already in anonymous namespace, no need for 'static'

Change-Id: I4278b9c371a8c1a74dbcdb87be434ec7f4fa3752

scilab/modules/scicos/src/cpp/view_scilab/LinkAdapter.cpp
scilab/modules/scicos/src/cpp/view_scilab/LinkAdapter.hxx
scilab/modules/scicos/src/cpp/view_scilab/ModelAdapter.cpp
scilab/modules/scicos/src/cpp/view_scilab/ports_management.hxx

index 91d237f..0f768f0 100644 (file)
@@ -290,7 +290,7 @@ struct ct
     }
 };
 
-static types::Double* getLinkEnd(const LinkAdapter& adaptor, const Controller& controller, object_properties_t end)
+types::Double* getLinkEnd(const LinkAdapter& adaptor, const Controller& controller, const object_properties_t end)
 {
     model::Link* adaptee = adaptor.getAdaptee();
 
@@ -374,7 +374,7 @@ enum startOrEnd
  * Connectivity is ensured if 'port' is of the desired type or if either of the concerned blocks is a split block,
  * because they are connectable to anything
  */
-static bool checkConnectivity(const int neededType, const ScicosID port, const ScicosID blk1, Controller& controller)
+bool checkConnectivity(const int neededType, const ScicosID port, const ScicosID blk1, Controller& controller)
 {
     int portKind;
     controller.getObjectProperty(port, PORT, PORT_KIND, portKind);
@@ -399,7 +399,7 @@ static bool checkConnectivity(const int neededType, const ScicosID port, const S
     return true;
 }
 
-static void setLinkEnd(ScicosID id, Controller& controller, object_properties_t end, const std::vector<double>& v)
+void setLinkEnd(const ScicosID id, Controller& controller, const object_properties_t end, const std::vector<double>& v)
 {
 
     ScicosID from;
@@ -806,7 +806,7 @@ std::vector<double> LinkAdapter::getFrom() const
     return from_content;
 }
 
-bool LinkAdapter::setFrom(ScicosID id, const std::vector<double>& v, Controller& controller, bool model_level)
+bool LinkAdapter::setFrom(const ScicosID id, const std::vector<double>& v, Controller& controller, const bool model_level)
 {
     if (v.size() >= 2)
     {
@@ -851,7 +851,7 @@ std::vector<double> LinkAdapter::getTo() const
     return to_content;
 }
 
-bool LinkAdapter::setTo(ScicosID id, const std::vector<double>& v, Controller& controller, bool model_level)
+bool LinkAdapter::setTo(const ScicosID id, const std::vector<double>& v, Controller& controller, const bool model_level)
 {
     if (v.size() >= 2)
     {
index 7aa8d02..5545b94 100644 (file)
@@ -40,9 +40,9 @@ public:
     types::InternalType* clone();
 
     std::vector<double> getFrom() const;
-    bool setFrom(ScicosID id, const std::vector<double>& v, Controller& controller, bool model_level = true);
+    bool setFrom(const ScicosID id, const std::vector<double>& v, Controller& controller, const bool model_level = true);
     std::vector<double> getTo() const;
-    bool setTo(ScicosID id, const std::vector<double>& v, Controller& controller, bool model_level = true);
+    bool setTo(const ScicosID id, const std::vector<double>& v, Controller& controller, const bool model_level = true);
 private:
     std::vector<double> from_content;
     std::vector<double> to_content;
index 6c6c17a..f4b3de5 100644 (file)
@@ -717,11 +717,11 @@ struct odstate
  * When setting a diagram in 'rpar', the Superblock's ports must be consistent with the "port blocks" inside it.
  * By "port blocks", we mean IN_f, OUT_f, CLKIN_f, CLKOUT_f, CLKINV_f, CLKOUTV_f, INIMPL_f and OUTIMPL_f.
  */
-static bool setInnerBlocksRefs(ModelAdapter& adaptor, std::vector<ScicosID>& children, Controller& controller)
+bool setInnerBlocksRefs(ModelAdapter& adaptor, const std::vector<ScicosID>& children, Controller& controller)
 {
     model::Block* adaptee = adaptor.getAdaptee();
 
-    for (std::vector<ScicosID>::iterator it = children.begin(); it != children.end(); ++it)
+    for (std::vector<ScicosID>::const_iterator it = children.begin(); it != children.end(); ++it)
     {
         if (controller.getObject(*it)->kind() == BLOCK) // Rule out Annotations and Links
         {
@@ -889,7 +889,7 @@ struct rpar
     }
 };
 
-static double toDouble(const int a)
+double toDouble(const int a)
 {
     return static_cast<double>(a);
 }
index 226963a..ed7ee41 100644 (file)
@@ -40,7 +40,7 @@ namespace view_scilab
  * Return a Scilab encoded value for a property.
  */
 template<typename Adaptor, object_properties_t p>
-types::InternalType* get_ports_property(const Adaptor& adaptor, object_properties_t port_kind, const Controller& controller)
+types::InternalType* get_ports_property(const Adaptor& adaptor, const object_properties_t port_kind, const Controller& controller)
 {
     model::Block* adaptee = adaptor.getAdaptee();
 
@@ -154,7 +154,7 @@ types::InternalType* get_ports_property(const Adaptor& adaptor, object_propertie
  * \note this method will ignore or return false if one of the ports does not exist, depending on the property setted.
  */
 template<typename Adaptor, object_properties_t p>
-bool set_ports_property(const Adaptor& adaptor, object_properties_t port_kind, Controller& controller, types::InternalType* v)
+bool set_ports_property(const Adaptor& adaptor, const object_properties_t port_kind, Controller& controller, types::InternalType* v)
 {
     model::Block* adaptee = adaptor.getAdaptee();
 
@@ -322,7 +322,7 @@ bool set_ports_property(const Adaptor& adaptor, object_properties_t port_kind, C
  * \return true on success, false otherwise
  */
 template<typename Adaptor, object_properties_t p>
-inline bool fillNewPorts(std::vector<int>& newPorts, const std::vector<ScicosID>& children, double* d)
+inline bool fillNewPorts(std::vector<int>& newPorts, const std::vector<ScicosID>& children, const double* d)
 {
     for (std::vector<int>::iterator it = newPorts.begin(); it != newPorts.end(); ++it, ++d)
     {
@@ -353,7 +353,7 @@ inline bool fillNewPorts(std::vector<int>& newPorts, const std::vector<ScicosID>
  * \param deletedObjects trash used to delete objects
  */
 template<typename Adaptor, object_properties_t p>
-inline bool updateNewPort(ScicosID oldPort, int newPort, Controller& controller,
+inline bool updateNewPort(const ScicosID oldPort, int newPort, Controller& controller,
                           std::vector<ScicosID>& children, std::vector<ScicosID>& deletedObjects)
 {
     if (p == CONNECTED_SIGNALS)
@@ -439,7 +439,7 @@ inline bool updateNewPort(ScicosID oldPort, int newPort, Controller& controller,
  * \return true on success, false otherwise
  */
 template<typename Adaptor, object_properties_t p>
-inline bool addNewPort(ScicosID newPortID, int newPort, const std::vector<ScicosID>& children, Controller& controller)
+inline bool addNewPort(const ScicosID newPortID, int newPort, const std::vector<ScicosID>& children, Controller& controller)
 {
     bool status = true;
     if (p == CONNECTED_SIGNALS)
@@ -485,7 +485,7 @@ inline bool addNewPort(ScicosID newPortID, int newPort, const std::vector<Scicos
  * Create ports if needed, remove ports if needed and set a default property on each port.
  */
 template<typename Adaptor, object_properties_t p>
-bool update_ports_property(const Adaptor& adaptor, object_properties_t port_kind,  Controller& controller, types::InternalType* v)
+bool update_ports_property(const Adaptor& adaptor, const object_properties_t port_kind, Controller& controller, types::InternalType* v)
 {
     model::Block* adaptee = adaptor.getAdaptee();
 
@@ -620,7 +620,6 @@ bool update_ports_property(const Adaptor& adaptor, object_properties_t port_kind
     return true;
 }
 
-
 } /* namespace view_scilab */
 } /* namespace org_scilab_modules_scicos */