Xcos MVC: use ScicosID() instead of 0 93/18493/3
Paul Bignier [Fri, 19 Aug 2016 09:15:38 +0000 (11:15 +0200)]
Change-Id: I05ff2bd1d907b8d04a7718c84f0c5b817b8179ea

scilab/modules/scicos/src/cpp/Controller.cpp
scilab/modules/scicos/src/cpp/Model.cpp
scilab/modules/scicos/src/cpp/view_scilab/DiagramAdapter.cpp
scilab/modules/scicos/src/cpp/view_scilab/LinkAdapter.cpp
scilab/modules/scicos/src/cpp/view_scilab/ModelAdapter.cpp

index 452315c..a20ac65 100644 (file)
@@ -214,7 +214,7 @@ unsigned Controller::referenceObject(const ScicosID uid) const
 void Controller::deleteObject(ScicosID uid)
 {
     // if this object is the empty uid, ignore it : is is not stored in the model
-    if (uid == 0)
+    if (uid == ScicosID())
     {
         return;
     }
@@ -311,7 +311,7 @@ void Controller::unlinkVector(ScicosID uid, kind_t k, object_properties_t uid_pr
 {
     ScicosID v;
     getObjectProperty(uid, k, uid_prop, v);
-    if (v != 0)
+    if (v != ScicosID())
     {
         auto o = getObject(v);
         if (o == nullptr)
@@ -338,7 +338,7 @@ void Controller::unlink(ScicosID uid, kind_t k, object_properties_t uid_prop, ob
     getObjectProperty(uid, k, uid_prop, v);
     for (const ScicosID id : v)
     {
-        if (id != 0)
+        if (id != ScicosID())
         {
             auto o = getObject(id);
             if (o == nullptr)
@@ -453,7 +453,7 @@ void Controller::deepClone(std::map<ScicosID, ScicosID>& mapped, ScicosID uid, S
     ScicosID v;
     getObjectProperty(uid, k, p, v);
 
-    ScicosID cloned = 0;
+    ScicosID cloned = ScicosID();
 
     std::map<ScicosID, ScicosID>::iterator it = mapped.find(v);
     if (it != mapped.end())
@@ -464,24 +464,24 @@ void Controller::deepClone(std::map<ScicosID, ScicosID>& mapped, ScicosID uid, S
     {
         if (cloneIfNotFound)
         {
-            if (v != 0)
+            if (v != ScicosID())
             {
                 cloned = cloneObject(mapped, v, true, true);
             }
             else
             {
-                cloned = 0;
+                cloned = ScicosID();
             }
         }
         else
         {
-            cloned = 0;
+            cloned = ScicosID();
         }
     }
 
     setObjectProperty(clone, k, p, cloned);
     // When cloning a Link, connect both extremities together
-    if ((p == SOURCE_PORT || p == DESTINATION_PORT) && cloned != 0)
+    if ((p == SOURCE_PORT || p == DESTINATION_PORT) && cloned != ScicosID())
     {
         setObjectProperty(cloned, PORT, CONNECTED_SIGNALS, clone);
     }
@@ -497,7 +497,7 @@ void Controller::deepCloneVector(std::map<ScicosID, ScicosID>& mapped, ScicosID
 
     for (const ScicosID & id : v)
     {
-        if (id == 0)
+        if (id == ScicosID())
         {
             // Deleted Block, the cloning is done at Adapter level
             cloned.push_back(id);
@@ -529,7 +529,7 @@ void Controller::deepCloneVector(std::map<ScicosID, ScicosID>& mapped, ScicosID
         {
             if (cloneIfNotFound)
             {
-                if (id != 0)
+                if (id != ScicosID())
                 {
                     cloned.push_back(cloneObject(mapped, id, true, true));
                 }
index f81eaaa..33d9e3c 100644 (file)
@@ -71,6 +71,8 @@ ScicosID Model::createObject(kind_t k)
         case PORT:
             o = new model::Port();
             break;
+        default:
+            return ScicosID();
     }
 
     /*
index 48a6661..a81f90e 100644 (file)
@@ -87,7 +87,7 @@ struct objs
         int link_i = 0;
         for (int i = 0; i < static_cast<int>(children.size()); ++i)
         {
-            if (children[i] == 0ll)
+            if (children[i] == ScicosID())
             {
                 // Deleted adapter
                 types::MList* deleted = new types::MList();
@@ -404,7 +404,7 @@ struct objs
             std::sort(oldDiagramChildren.begin(), oldDiagramChildren.end());
             for (const ScicosID id : diagramChildren)
             {
-                if (id != 0 && !std::binary_search(oldDiagramChildren.begin(), oldDiagramChildren.end(), id))
+                if (id != ScicosID() && !std::binary_search(oldDiagramChildren.begin(), oldDiagramChildren.end(), id))
                 {
                     auto o = controller.getObject(id);
                     controller.setObjectProperty(o->id(), o->kind(), PARENT_DIAGRAM, adaptee->id());
@@ -416,7 +416,7 @@ struct objs
             std::sort(diagramChildren.begin(), diagramChildren.end());
             for (const ScicosID id : oldDiagramChildren)
             {
-                if (id != 0 && !std::binary_search(diagramChildren.begin(), diagramChildren.end(), id))
+                if (id != ScicosID() && !std::binary_search(diagramChildren.begin(), diagramChildren.end(), id))
                 {
                     auto o = controller.getObject(id);
                     controller.setObjectProperty(o->id(), o->kind(), PARENT_DIAGRAM, ScicosID());
index b04439b..a44fd1c 100644 (file)
@@ -315,7 +315,7 @@ link_t getLinkEnd(const LinkAdapter& adaptor, const Controller& controller, cons
 
     ScicosID endID;
     controller.getObjectProperty(adaptee, LINK, end, endID);
-    if (endID != 0)
+    if (endID != ScicosID())
     {
         ScicosID sourceBlock;
         controller.getObjectProperty(endID, PORT, SOURCE_BLOCK, sourceBlock);
@@ -326,12 +326,12 @@ link_t getLinkEnd(const LinkAdapter& adaptor, const Controller& controller, cons
         controller.getObjectProperty(adaptee, LINK, PARENT_BLOCK, parent);
         std::vector<ScicosID> children;
         // Added to a superblock
-        if (parent == 0)
+        if (parent == ScicosID())
         {
             // Added to a diagram
             controller.getObjectProperty(adaptee, LINK, PARENT_DIAGRAM, parent);
             parentKind = DIAGRAM;
-            if (parent == 0)
+            if (parent == ScicosID())
             {
                 return ret;
             }
@@ -431,12 +431,12 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
         default:
             return;
     }
-    ScicosID unconnected = 0;
+    ScicosID unconnected = ScicosID();
 
     if (v.block == 0 || v.port == 0)
     {
         // We want to set an empty link
-        if (concernedPort == 0)
+        if (concernedPort == ScicosID())
         {
             // In this case, the link was already empty, do a dummy call to display the console status.
             controller.setObjectProperty(id, LINK, end, concernedPort);
@@ -453,7 +453,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
     ScicosID parentDiagram;
     controller.getObjectProperty(id, LINK, PARENT_DIAGRAM, parentDiagram);
     std::vector<ScicosID> children;
-    if (parentDiagram != 0)
+    if (parentDiagram != ScicosID())
     {
         // Adding to a diagram
         controller.getObjectProperty(parentDiagram, DIAGRAM, CHILDREN, children);
@@ -462,7 +462,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
     {
         ScicosID parentBlock;
         controller.getObjectProperty(id, LINK, PARENT_BLOCK, parentBlock);
-        if (parentBlock != 0)
+        if (parentBlock != ScicosID())
         {
             // Adding to a superblock
             controller.getObjectProperty(parentBlock, BLOCK, CHILDREN, children);
@@ -484,7 +484,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
     }
     ScicosID blkID = children[v.block - 1];
 
-    if (blkID == 0)
+    if (blkID == ScicosID())
     {
         // Deleted Block
         return;
@@ -513,7 +513,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
 
         if (v.kind == Start)
         {
-            if (otherPort != 0)
+            if (otherPort != ScicosID())
             {
                 if (!checkConnectivity(PORT_EIN, otherPort, blkID, controller))
                 {
@@ -525,7 +525,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
         }
         else
         {
-            if (otherPort != 0)
+            if (otherPort != ScicosID())
             {
                 if (!checkConnectivity(PORT_EOUT, otherPort, blkID, controller))
                 {
@@ -548,7 +548,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
         {
             if (v.kind == Start)
             {
-                if (otherPort != 0)
+                if (otherPort != ScicosID())
                 {
                     if (!checkConnectivity(PORT_IN, otherPort, blkID, controller))
                     {
@@ -560,7 +560,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
             }
             else
             {
-                if (otherPort != 0)
+                if (otherPort != ScicosID())
                 {
                     if (!checkConnectivity(PORT_OUT, otherPort, blkID, controller))
                     {
@@ -616,7 +616,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
     }
 
     // Disconnect the old port if it was connected. After that, concernedPort will be reused to designate the new port
-    if (concernedPort != 0)
+    if (concernedPort != ScicosID())
     {
         controller.setObjectProperty(concernedPort, PORT, CONNECTED_SIGNALS, unconnected);
     }
@@ -677,7 +677,7 @@ void setLinkEnd(const ScicosID id, Controller& controller, const object_properti
     }
     ScicosID oldLink;
     controller.getObjectProperty(concernedPort, PORT, CONNECTED_SIGNALS, oldLink);
-    if (oldLink != 0)
+    if (oldLink != ScicosID())
     {
         // Disconnect the old link
         controller.setObjectProperty(oldLink, LINK, end, unconnected);
@@ -894,7 +894,7 @@ void LinkAdapter::setFromInModel(const link_t& v, Controller& controller)
     ScicosID parentBlock;
     controller.getObjectProperty(getAdaptee()->id(), LINK, PARENT_BLOCK, parentBlock);
 
-    if (parentDiagram != 0 || parentBlock != 0)
+    if (parentDiagram != ScicosID() || parentBlock != ScicosID())
     {
         // If the Link has been added to a diagram, do the linking at model-level
         // If the provided values are wrong, the model is not updated but the info is stored in the Adapter for future attempts
@@ -921,7 +921,7 @@ void LinkAdapter::setToInModel(const link_t& v, Controller& controller)
     ScicosID parentBlock;
     controller.getObjectProperty(getAdaptee()->id(), LINK, PARENT_BLOCK, parentBlock);
 
-    if (parentDiagram != 0 || parentBlock != 0)
+    if (parentDiagram != ScicosID() || parentBlock != ScicosID())
     {
         // If the Link has been added to a diagram, do the linking at model-level
         // If the provided values are wrong, the model is not updated but the info is stored in the Adapter for future attempts
index dedd591..6ca75cb 100644 (file)
@@ -764,7 +764,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)
         }