PVS: V519 A variable is assigned values twice successively.

This commit is contained in:
wmayer
2020-07-17 09:48:08 +02:00
parent 8a7b4a145b
commit e005972466
3 changed files with 12 additions and 8 deletions

View File

@@ -246,8 +246,10 @@ void GLImageBox::getDisplayedImageAreaSize(int &dx, int &dy)
dx = 0;
dy = 0;
}
dx = ibrx - itlx + 1;
dy = ibry - itly + 1;
else {
dx = ibrx - itlx + 1;
dy = ibry - itly + 1;
}
}
}

View File

@@ -316,9 +316,10 @@ private:
if(keepPlacement!=Py_None)
ocaf.setKeepPlacement(PyObject_IsTrue(keepPlacement));
ocaf.exportObjects(objs);
}else{
bool keepExplicitPlacement = objs.size() > 1;
keepExplicitPlacement = Standard_True;
}
else {
//bool keepExplicitPlacement = objs.size() > 1;
bool keepExplicitPlacement = Standard_True;
ExportOCAF ocaf(hDoc, keepExplicitPlacement);
// That stuff is exporting a list of selected objects into FreeCAD Tree
std::vector <TDF_Label> hierarchical_label;

View File

@@ -597,9 +597,10 @@ private:
if(keepPlacement!=Py_None)
ocaf.setKeepPlacement(PyObject_IsTrue(keepPlacement));
ocaf.exportObjects(objs);
} else {
bool keepExplicitPlacement = objs.size() > 1;
keepExplicitPlacement = Standard_True;
}
else {
//bool keepExplicitPlacement = objs.size() > 1;
bool keepExplicitPlacement = Standard_True;
ExportOCAFGui ocaf(hDoc, keepExplicitPlacement);
// That stuff is exporting a list of selected objects into FreeCAD Tree
std::vector <TDF_Label> hierarchical_label;