don't break the loop after interior ring while calculating perimeter:

the returned ptr's pointing to a wrong position whether we break the loop before reading all the points of all the rings.
qgis · Dec 16, 2012 · 923ab1b · 923ab1b · brushtyler · Dec 17, 2012
1 parent a2f4db6
commit 923ab1b
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/src/core/qgsdistancearea.cpp b/src/core/qgsdistancearea.cpp
@@ -543,16 +543,15 @@ unsigned char* QgsDistanceArea::measurePolygon( unsigned char* feature, double*
 
         if ( perimeter )
         {
-          *perimeter += measureLine( points );
+          if ( idx == 0 )
+          {
+            // exterior ring
+            *perimeter += measureLine( points );
+          }
         }
       }
 
       points.clear();
-
-      if ( !area )
-      {
-        break;
-      }
     }
   }
   catch ( QgsCsException &cse )