fix compiler warnings

FreeCAD · Sep 1, 2016 · e6575f6 · e6575f6
1 parent fa2b708
commit e6575f6
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Mod/Part/Gui/DlgExtrusion.cpp b/src/Mod/Part/Gui/DlgExtrusion.cpp
@@ -565,7 +565,7 @@ std::vector<App::DocumentObject*> DlgExtrusion::getShapesToExtrude() const
         throw Base::Exception("Document lost");
 
     std::vector<App::DocumentObject*> 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
@@ -236,7 +236,7 @@ std::vector<App::DocumentObject*> DlgRevolution::getShapesToRevolve() const
         throw Base::Exception("Document lost");
 
     std::vector<App::DocumentObject*> 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;