Xcos MVC: minor fix to specify long long constant 31/17231/3
Paul Bignier [Tue, 22 Sep 2015 07:59:42 +0000 (09:59 +0200)]
Change-Id: I687365235ab4f63dc60057f48bba8d8e27904067

scilab/modules/scicos/src/cpp/Controller.cpp
scilab/modules/scicos/src/cpp/Model.cpp
scilab/modules/scicos/src/cpp/model/Annotation.hxx
scilab/modules/scicos/src/cpp/model/Block.hxx
scilab/modules/scicos/src/cpp/model/Link.hxx
scilab/modules/scicos/src/cpp/model/Port.hxx
scilab/modules/scicos/src/cpp/view_scilab/DiagramAdapter.cpp
scilab/modules/scicos/src/cpp/view_scilab/ModelAdapter.cpp

index e6860e8..dd7ec65 100644 (file)
@@ -413,12 +413,12 @@ void Controller::deepCloneVector(std::map<ScicosID, ScicosID>& mapped, ScicosID
                 }
                 else
                 {
-                    cloned.push_back(0);
+                    cloned.push_back(ScicosID());
                 }
             }
             else
             {
-                cloned.push_back(0);
+                cloned.push_back(ScicosID());
             }
         }
     }
index 67fe62e..fc39e1f 100644 (file)
@@ -29,7 +29,7 @@ namespace org_scilab_modules_scicos
 {
 
 Model::Model() :
-    lastId(0), has_looped(false), allObjects()
+    lastId(ScicosID()), has_looped(false), allObjects()
 {
     std::vector<int> datatypeDefault (3, 1);
     datatypeDefault[0] = -1;
@@ -74,7 +74,7 @@ ScicosID Model::createObject(kind_t k)
      * Found the next unused id
      */
     lastId++;
-    if (lastId == 0)
+    if (lastId == ScicosID())
     {
         lastId++;
         has_looped = true;
@@ -91,7 +91,7 @@ ScicosID Model::createObject(kind_t k)
         {
             // try a valid ID
             lastId++;
-            if (lastId == 0)
+            if (lastId == ScicosID())
             {
                 lastId++;
 
index dad6b0e..064bba1 100644 (file)
@@ -28,8 +28,8 @@ namespace model
 class Annotation: public BaseObject
 {
 public:
-    Annotation(): BaseObject(ANNOTATION), m_parentDiagram(0), m_parentBlock(0),
-        m_description(std::string("Text")), m_font(std::string("2")), m_font_size(std::string("1")), m_relatedTo(0)
+    Annotation(): BaseObject(ANNOTATION), m_parentDiagram(ScicosID()), m_parentBlock(ScicosID()),
+        m_description(std::string("Text")), m_font(std::string("2")), m_font_size(std::string("1")), m_relatedTo(ScicosID())
     {
         std::vector<double> geom (4, 0);
         geom[2] = 2;
index e6e427a..fc56932 100644 (file)
@@ -117,9 +117,9 @@ struct Angle
 class Block: public BaseObject
 {
 public:
-    Block() : BaseObject(BLOCK), m_parentDiagram(0), m_interfaceFunction(), m_geometry(), m_angle(),
+    Block() : BaseObject(BLOCK), m_parentDiagram(ScicosID()), m_interfaceFunction(), m_geometry(), m_angle(),
         m_exprs(std::vector<double> (5, 0)), m_label(), m_style(), m_nzcross(std::vector<int> (1, 0)), m_nmode(std::vector<int> (1, 0)), m_equations(), m_uid(), m_sim(), m_in(), m_out(), m_ein(), m_eout(),
-        m_parameter(), m_state(), m_parentBlock(0), m_children(), m_portReference(0)
+        m_parameter(), m_state(), m_parentBlock(ScicosID()), m_children(), m_portReference(ScicosID())
     {
         // By default, 'm_exprs' takes the value of vec2var([]), which is {12, 2, 0, 0, 0}
         m_exprs[0] = 12;
index a6caf80..977563c 100644 (file)
@@ -38,7 +38,7 @@ enum link_kind_t
 class Link: public BaseObject
 {
 public:
-    Link() : BaseObject(LINK), m_parentDiagram(0), m_parentBlock(0), m_sourcePort(0), m_destinationPort(0), m_controlPoints(),
+    Link() : BaseObject(LINK), m_parentDiagram(ScicosID()), m_parentBlock(ScicosID()), m_sourcePort(ScicosID()), m_destinationPort(ScicosID()), m_controlPoints(),
         m_label(), m_thick(std::vector<double>(2)), m_color(1), m_linkKind(regular) {};
     Link(const Link& o) : BaseObject(LINK), m_parentDiagram(o.m_parentDiagram), m_parentBlock(o.m_parentBlock), m_sourcePort(o.m_sourcePort), m_destinationPort(o.m_destinationPort),
         m_controlPoints(o.m_controlPoints), m_label(o.m_label), m_thick(o.m_thick), m_color(o.m_color), m_linkKind(o.m_linkKind) {};
index f6bd13e..cc0be50 100644 (file)
@@ -33,8 +33,8 @@ enum portKind
 class Port: public BaseObject
 {
 public:
-    Port() : BaseObject(PORT), m_dataType(0), m_sourceBlock(0), m_kind(PORT_UNDEF), m_implicit(false),
-        m_style(), m_label(), m_firing(0), m_connectedSignals(std::vector<ScicosID> (1, 0)) {};
+    Port() : BaseObject(PORT), m_dataType(0), m_sourceBlock(ScicosID()), m_kind(PORT_UNDEF), m_implicit(false),
+        m_style(), m_label(), m_firing(0), m_connectedSignals(std::vector<ScicosID> (1, ScicosID())) {};
     Port(const Port& o) : BaseObject(PORT), m_dataType(o.m_dataType), m_sourceBlock(o.m_sourceBlock), m_kind(o.m_kind), m_implicit(o.m_implicit),
         m_style(o.m_style), m_label(o.m_label), m_firing(0), m_connectedSignals(o.m_connectedSignals) {};
     ~Port() = default;
index 05e434b..bbbf231 100644 (file)
@@ -141,7 +141,7 @@ struct objs
 
         /*
          * First pass on objects:
-         *  - store IDs if they exists and are valid ; 0ll otherwise
+         *  - store IDs if they exists and are valid ; ScicosID() otherwise
          *  - store all the links to update link connections later
          *  - store all the valid mlist content ('Text' content)
          *  - store all the deleted mlist content ('Deleted' content)
@@ -213,14 +213,14 @@ struct objs
                 if (header->get(0) == Deleted)
                 {
                     deletedAsMListIndices.push_back(i);
-                    diagramChildren.push_back(0ll);
+                    diagramChildren.push_back(ScicosID());
                     diagramChildrenKind.push_back(ANNOTATION);
                 }
                 else if (header->get(0) == TextSharedTypeStr)
                 {
                     textAsMListIndices.push_back(i);
                     // will be filled later
-                    diagramChildren.push_back(0ll);
+                    diagramChildren.push_back(ScicosID());
                     diagramChildrenKind.push_back(ANNOTATION);
                 }
                 else if (header->get(0) == BlockSharedTypeStr)
@@ -269,7 +269,7 @@ struct objs
 
                 deletedAsMListIndices.push_back(i);
                 // Mark deleted objects with value '0'
-                diagramChildren.push_back(0ll);
+                diagramChildren.push_back(ScicosID());
                 diagramChildrenKind.push_back(ANNOTATION);
             }
             else
index 275c6b2..1b7da8c 100644 (file)
@@ -750,7 +750,7 @@ bool setInnerBlocksRefs(ModelAdapter& adaptor, const std::vector<ScicosID>& chil
 
     for (std::vector<ScicosID>::const_iterator it = children.begin(); it != children.end(); ++it)
     {
-        if (*it == 0ll)
+        if (*it == ScicosID())
         {
             continue; // Rule out mlists (Deleted or Annotations)
         }
@@ -939,7 +939,7 @@ struct rpar
             if (diagramChildren.empty())
             {
                 // bug_12998: If inserting an empty diagram in 'rpar', simulate an empty object
-                diagramChildren.push_back(0ll);
+                diagramChildren.push_back(ScicosID());
             }
             std::vector<ScicosID> oldDiagramChildren;
             controller.getObjectProperty(adaptor.getAdaptee()->id(), BLOCK, CHILDREN, oldDiagramChildren);
@@ -992,7 +992,7 @@ struct rpar
             if (diagramChildren.empty())
             {
                 // bug_12998: If inserting an empty diagram in 'rpar', simulate an empty object
-                diagramChildren.push_back(0ll);
+                diagramChildren.push_back(ScicosID());
             }
             std::vector<ScicosID> oldDiagramChildren;
             controller.getObjectProperty(adaptor.getAdaptee()->id(), BLOCK, CHILDREN, oldDiagramChildren);