kindred/create
1
1
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects 9 Releases 2 Wiki Activity

PartDesign: Use PyObject_IsTrue in combination with conditional ternary operator

Browse Source
This commit is contained in:
marioalexis
2022-05-17 12:55:50 -03:00
committed by Chris Hennes
parent b10f73b571
commit 15923813d4
1 changed files with 12 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
src/Mod/PartDesign/Gui/ViewProviderPyImp.cpp
View File

@@ -49,26 +49,26 @@ int ViewProviderPy::setCustomAttributes(const char* , PyObject *)
PyObject* ViewProviderPy::setBodyMode(PyObject* args)
{
PartDesignGui::ViewProvider* base = getViewProviderPtr();
PyObject* b_mode;
if(PyArg_ParseTuple(args, "O!", &PyBool_Type, &b_mode)){
base->setBodyMode(PyObject_IsTrue(b_mode));
return Py::new_reference_to(Py::None());
if (!PyArg_ParseTuple(args, "O!", &PyBool_Type, &b_mode)) {
return nullptr;
};
return nullptr; //error
PartDesignGui::ViewProvider* base = getViewProviderPtr();
base->setBodyMode(PyObject_IsTrue(b_mode) ? true : false);
return Py::new_reference_to(Py::None());
}
PyObject* ViewProviderPy::makeTemporaryVisible(PyObject* args)
{
PartDesignGui::ViewProvider* base = getViewProviderPtr();
PyObject* b_vis;
if(PyArg_ParseTuple(args, "O!", &PyBool_Type, &b_vis)){
base->makeTemporaryVisible(PyObject_IsTrue(b_vis));
return Py::new_reference_to(Py::None());
if (!PyArg_ParseTuple(args, "O!", &PyBool_Type, &b_vis)) {
return nullptr;
};
return nullptr; //error
PartDesignGui::ViewProvider* base = getViewProviderPtr();
base->makeTemporaryVisible(PyObject_IsTrue(b_vis) ? true : false);
return Py::new_reference_to(Py::None());
}