Fix return none in cmaboss

sysbio-curie · Nov 15, 2024 · 5670b7a · 5670b7a
1 parent cf1daa5
commit 5670b7a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/engine/python/cmaboss/maboss_net.cpp b/engine/python/cmaboss/maboss_net.cpp
@@ -151,7 +151,7 @@ PyObject* cMaBoSSNetwork_setOutput(cMaBoSSNetworkObject* self, PyObject *args)
       node->isInternal(true);
     }
   }
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cMaBoSSNetwork_getOutput(cMaBoSSNetworkObject* self) 
@@ -181,7 +181,7 @@ PyObject* cMaBoSSNetwork_setObservedGraphNode(cMaBoSSNetworkObject* self, PyObje
       node->inGraph(false);
     }
   }
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cMaBoSSNetwork_getObservedGraphNode(cMaBoSSNetworkObject* self, PyObject *args) 
@@ -213,7 +213,7 @@ PyObject* cMaBoSSNetwork_addNode(cMaBoSSNetworkObject* self, PyObject *args)
     return NULL;
   }
 
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cMaBoSSNetwork_setIState(cMaBoSSNetworkObject* self, PyObject *args) 

diff --git a/engine/python/cmaboss/maboss_sim.cpp b/engine/python/cmaboss/maboss_sim.cpp
@@ -240,7 +240,7 @@ PyObject* cMaBoSSSim_check(cMaBoSSSimObject* self, PyObject *args, PyObject* kwa
     PyErr_SetString(PyBNException, e.getMessage().c_str());
     return NULL;
   }
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cMaBoSSSim_get_logical_rules(cMaBoSSSimObject* self, PyObject *args, PyObject* kwargs) {

diff --git a/engine/python/cmaboss/popmaboss_net.cpp b/engine/python/cmaboss/popmaboss_net.cpp
@@ -143,8 +143,7 @@ PyObject* cPopMaBoSSNetwork_getDeathRate(cPopMaBoSSNetworkObject* self)
 {
   const Expression* death_rate = self->network->getDeathRate();
   if (death_rate == NULL) {
-    Py_INCREF(Py_None);
-    return Py_None;
+    Py_RETURN_NONE;
   }
 
   PyObject* death_rate_str = PyUnicode_FromString(
@@ -177,8 +176,7 @@ PyObject* cPopMaBoSSNetwork_setDeathRate(cPopMaBoSSNetworkObject* self, PyObject
     return NULL;
   }
 
-  Py_INCREF(Py_None);
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cPopMaBoSSNetwork_setOutput(cPopMaBoSSNetworkObject* self, PyObject *args) 
@@ -195,7 +193,7 @@ PyObject* cPopMaBoSSNetwork_setOutput(cPopMaBoSSNetworkObject* self, PyObject *a
       node->isInternal(true);
     }
   }
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cPopMaBoSSNetwork_getOutput(cPopMaBoSSNetworkObject* self) 
@@ -252,8 +250,7 @@ PyObject* cPopMaBoSSNetwork_addDivisionRule(cPopMaBoSSNetworkObject* self, PyObj
     return NULL;
   }
 
-  Py_INCREF(Py_None);
-  return Py_None;
+  Py_RETURN_NONE;
 }
 
 PyObject* cPopMaBoSSNetwork_getDivisionRules(cPopMaBoSSNetworkObject* self)