From e6575f65f98927d9ae4d894d8017ddb574a6a905 Mon Sep 17 00:00:00 2001 From: wmayer Date: Thu, 1 Sep 2016 12:01:22 +0200 Subject: [PATCH] fix compiler warnings --- src/Mod/Part/Gui/DlgExtrusion.cpp | 2 +- src/Mod/Part/Gui/DlgRevolution.cpp | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/Mod/Part/Gui/DlgExtrusion.cpp b/src/Mod/Part/Gui/DlgExtrusion.cpp index ae26a67e6..ce1f616d8 100644 --- a/src/Mod/Part/Gui/DlgExtrusion.cpp +++ b/src/Mod/Part/Gui/DlgExtrusion.cpp @@ -565,7 +565,7 @@ std::vector DlgExtrusion::getShapesToExtrude() const throw Base::Exception("Document lost"); std::vector objects; - for(size_t i = 0 ; i < items.size() ; i++){ + for (int i = 0; i < items.size(); i++) { App::DocumentObject* obj = doc->getObject(items[i]->data(0, Qt::UserRole).toString().toLatin1()); if (!obj) throw Base::Exception("Object not found"); diff --git a/src/Mod/Part/Gui/DlgRevolution.cpp b/src/Mod/Part/Gui/DlgRevolution.cpp index a1db0402d..8027801b2 100644 --- a/src/Mod/Part/Gui/DlgRevolution.cpp +++ b/src/Mod/Part/Gui/DlgRevolution.cpp @@ -236,7 +236,7 @@ std::vector DlgRevolution::getShapesToRevolve() const throw Base::Exception("Document lost"); std::vector objects; - for(size_t i = 0 ; i < items.size() ; i++){ + for (int i = 0; i < items.size(); i++) { App::DocumentObject* obj = doc->getObject(items[i]->data(0, Qt::UserRole).toString().toLatin1()); if (!obj) throw Base::Exception("Object not found"); @@ -293,7 +293,7 @@ bool DlgRevolution::validate() //check angle if (!axisLinkHasAngle){ - if (fabs(this->getAngle() / 180.0 * M_PI) < Precision::Angular()){ + if (fabs(this->getAngle() / 180.0 * M_PI) < Precision::Angular()) { QMessageBox::critical(this, windowTitle(), tr("Revolution angle span is zero. It must be non-zero.")); ui->angle->setFocus(); @@ -522,7 +522,7 @@ void DlgRevolution::autoSolid() ShapeExtend_Explorer xp; Handle_TopTools_HSequenceOfShape leaves = xp.SeqFromCompound(sh, /*recursive= */Standard_True); int cntClosedWires = 0; - for(int i = 0 ; i < leaves->Length() ; i++){ + for (int i = 0; i < leaves->Length(); i++) { const TopoDS_Shape &leaf = leaves->Value(i+1); if (leaf.IsNull()) return;