From: Jean-Baptiste Mardelle <jb@kdenlive.org>
Date: Sun, 29 May 2011 17:41:00 +0000 (+0000)
Subject: small cleanup
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=5762e7870bec80b9c61d5486065c0ebb33ddb1fb;p=kdenlive

small cleanup

svn path=/trunk/kdenlive/; revision=5618
---

diff --git a/src/kthumb.cpp b/src/kthumb.cpp
index 6af26e4f..1a953d5a 100644
--- a/src/kthumb.cpp
+++ b/src/kthumb.cpp
@@ -203,7 +203,7 @@ uint KThumb::imageVariance(QImage image )
     uint bytes = image.numBytes();
     uint STEPS = bytes/2;
     QVarLengthArray<uchar> pivot(STEPS);
-    uchar *bits=image.bits();
+    const uchar *bits=image.bits();
     // First pass: get pivots and taking average
     for( uint i=0; i<STEPS ; i++ ){
         pivot[i] = bits[2 * i];
diff --git a/thumbnailer/westleypreview.cpp b/thumbnailer/westleypreview.cpp
index 0ce57323..a6878752 100644
--- a/thumbnailer/westleypreview.cpp
+++ b/thumbnailer/westleypreview.cpp
@@ -126,17 +126,18 @@ uint MltPreview::imageVariance(QImage image)
     uint STEPS = bytes / 2;
     QVarLengthArray<uchar> pivot(STEPS);
     kDebug(DBG_AREA) << "Using " << STEPS << " steps\n";
-    uchar *bits = image.bits();
+    const uchar *bits=image.bits();
     // First pass: get pivots and taking average
-    for (uint i = 0; i < STEPS ; i++) {
-        pivot[i] = bits[i*(bytes/STEPS)];
-        avg += pivot[i];
+    for( uint i=0; i<STEPS ; i++ ){
+        pivot[i] = bits[2 * i];
+        avg+=pivot.at(i);
     }
-    avg = avg / STEPS;
+    avg=avg/STEPS;
     // Second Step: calculate delta (average?)
-    for (uint i = 0; i < STEPS; i++) {
-        int curdelta = abs(int(avg - pivot[i]));
-        delta += curdelta;
+    for (uint i=0; i<STEPS; i++)
+    {
+        int curdelta=abs(int(avg - pivot.at(i)));
+        delta+=curdelta;
     }
     return delta / STEPS;
 }