lavfi/mp=decimate: fix off-by-one logic in diff_C() x loop
authorStefano Sabatini <stefasab@gmail.com>
Wed, 29 Aug 2012 11:08:49 +0000 (13:08 +0200)
committerStefano Sabatini <stefasab@gmail.com>
Fri, 31 Aug 2012 14:04:12 +0000 (16:04 +0200)
Set x offset values in the range 0-7, rather than in the range 8-1.

The y loop is changed accordingly, to avoid confusion.

This also fixes output difference with the new pending native decimate
filter.

libavfilter/libmpcodecs/vf_decimate.c

index 1fd7bce..2d2aaf0 100644 (file)
@@ -82,8 +82,8 @@ static int diff_MMX(unsigned char *old, unsigned char *new, int os, int ns)
 static int diff_C(unsigned char *old, unsigned char *new, int os, int ns)
 {
     int x, y, d=0;
-    for (y = 8; y; y--) {
-        for (x = 8; x; x--) {
+    for (y = 0; y < 8; y++) {
+        for (x = 0; x < 8; x++) {
             d += abs(new[x] - old[x]);
         }
         new += ns;