Fix off-by-one calculation of element sum (raster stats).

git-svn-id: http://svn.osgeo.org/qgis/trunk@14069 c8812cc2-4d05-0410-92ff-de0c093fc19c
qgis · Aug 12, 2010 · 2843a93 · 2843a93
1 parent 5dfa6e6
commit 2843a93
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/core/raster/qgsrasterlayer.cpp b/src/core/raster/qgsrasterlayer.cpp
@@ -824,14 +824,15 @@ const QgsRasterBandStats QgsRasterLayer::bandStatistics( int theBandNo )
             continue; // NULL
           }
 
+          myRasterBandStats.sum += myValue;          
+          ++myRasterBandStats.elementCount;
           //only use this element if we have a non null element
           if ( myFirstIterationFlag )
           {
             //this is the first iteration so initialise vars
             myFirstIterationFlag = false;
             myRasterBandStats.minimumValue = myValue;
             myRasterBandStats.maximumValue = myValue;
-            ++myRasterBandStats.elementCount;
           }               //end of true part for first iteration check
           else
           {
@@ -844,9 +845,6 @@ const QgsRasterBandStats QgsRasterLayer::bandStatistics( int theBandNo )
             {
               myRasterBandStats.maximumValue = myValue;
             }
-
-            myRasterBandStats.sum += myValue;
-            ++myRasterBandStats.elementCount;
           }               //end of false part for first iteration check
         }
       }