diff --git a/src/Mod/Part/App/ExtrusionHelper.cpp b/src/Mod/Part/App/ExtrusionHelper.cpp index 42989e494b..b4503f6641 100644 --- a/src/Mod/Part/App/ExtrusionHelper.cpp +++ b/src/Mod/Part/App/ExtrusionHelper.cpp @@ -252,13 +252,13 @@ void ExtrusionHelper::makeDraft(const TopoDS_Shape& shape, std::vector::iterator isInnerWireIterator = isInnerWire.begin(); std::vector::iterator isInnerWireIteratorLoop; for (auto itOuter = shells.begin(); itOuter != shells.end(); ++itOuter) { - if (*isInnerWireIterator == true) { + if (*isInnerWireIterator) { ++isInnerWireIterator; continue; } isInnerWireIteratorLoop = isInnerWire.begin(); for (auto itInner = shells.begin(); itInner != shells.end(); ++itInner) { - if (itOuter == itInner || *isInnerWireIteratorLoop == false) { + if (itOuter == itInner || !*isInnerWireIteratorLoop) { ++isInnerWireIteratorLoop; continue; } @@ -330,7 +330,7 @@ void ExtrusionHelper::checkInnerWires(std::vector& isInnerWire, const gp_D // if nothing happens, a prism can be cancelled from the checklist for (auto itOuter = prisms.begin(); itOuter != prisms.end(); ++itOuter) { ++outer; - if (*toCheckIterator == false) { + if (!*toCheckIterator) { ++isInnerWireIterator; ++toCheckIterator; continue; @@ -338,7 +338,7 @@ void ExtrusionHelper::checkInnerWires(std::vector& isInnerWire, const gp_D auto toCheckIteratorInner = checklist.begin(); bool saveIsInnerWireIterator = *isInnerWireIterator; for (auto itInner = prisms.begin(); itInner != prisms.end(); ++itInner) { - if (itOuter == itInner || *toCheckIteratorInner == false) { + if (itOuter == itInner || !*toCheckIteratorInner) { ++toCheckIteratorInner; continue; } diff --git a/src/Mod/Part/Gui/DlgBooleanOperation.cpp b/src/Mod/Part/Gui/DlgBooleanOperation.cpp index 1392217f45..3fe88c07a8 100644 --- a/src/Mod/Part/Gui/DlgBooleanOperation.cpp +++ b/src/Mod/Part/Gui/DlgBooleanOperation.cpp @@ -60,7 +60,7 @@ namespace PartGui { void setData (int column, int role, const QVariant & value) { QTreeWidgetItem::setData(column, role, value); - if (role == Qt::CheckStateRole && value.toBool() == true) { + if (role == Qt::CheckStateRole && value.toBool()) { QTreeWidget* tree = this->treeWidget(); if (!tree) return;