code review: move transaction to outside from loop, fix several bugs, fix whitespaces

2014-08-05 00:33:51 +02:00
parent c4c04e3143
commit 3924b63759
4 changed files with 222 additions and 213 deletions
--- a/src/Mod/Sketcher/App/SketchObject.cpp
+++ b/src/Mod/Sketcher/App/SketchObject.cpp
@@ -1633,11 +1633,11 @@ void SketchObject::getGeoVertexIndex(int VertexId, int &GeoId, PointPos &PosId)

 int SketchObject::getVertexIndexGeoPos(int GeoId, PointPos PosId) const
 {
-    
-    for(int i=0;i<VertexId2GeoId.size();i++)
-      if(VertexId2GeoId[i]==GeoId && VertexId2PosId[i]==PosId)
-	 return i;
-  
+    for(int i=0;i<VertexId2GeoId.size();i++) {
+        if(VertexId2GeoId[i]==GeoId && VertexId2PosId[i]==PosId)
+            return i;
+    }
+
    return -1;
 }