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

brushtyler · brushtyler · commit 923ab1bf6cd7 · 2012-12-17T00:52:12.000+01:00
the returned ptr's pointing to a wrong position whether we break the loop before reading all the points of all the rings.
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 )

Original file line number	Diff line number	Diff line change
`@@ -543,16 +543,15 @@ unsigned char* QgsDistanceArea::measurePolygon( unsigned char* feature, double*`
`543`	`543`
`544`	`544`	`if ( perimeter )`
`545`	`545`	`{`
`546`		`- *perimeter += measureLine( points );`
	`546`	`+ if ( idx == 0 )`
	`547`	`+ {`
	`548`	`+ // exterior ring`
	`549`	`+ *perimeter += measureLine( points );`
	`550`	`+ }`
`547`	`551`	`}`
`548`	`552`	`}`
`549`	`553`
`550`	`554`	`points.clear();`
`551`		`-`
`552`		`- if ( !area )`
`553`		`- {`
`554`		`- break;`
`555`		`- }`
`556`	`555`	`}`
`557`	`556`	`}`
`558`	`557`	`catch ( QgsCsException &cse )`