Changeset 3556Tweet

Timestamp:

07/19/09 22:37:12 (4 years ago)

Author:

pterjan

Message:

• When blitting, only add dirty rectangles on changed lines, or on changed chars if we have a mask

Location:

libcaca/trunk

Files:

• caca/string.c (modified) (2 diffs)
• tests/dirty.cpp (modified) (2 diffs)

libcaca/trunk/caca/string.c

@@ -449 +449 @@
                     continue;
 
-                dst->chars[dstix + i] = src->chars[srcix + i];
-                dst->attrs[dstix + i] = src->attrs[srcix + i];
+                if(dst->chars[dstix + i] != src->chars[srcix + i] ||
+                   dst->attrs[dstix + i] != src->attrs[srcix + i])
+                {
+                    dst->chars[dstix + i] = src->chars[srcix + i];
+                    dst->attrs[dstix + i] = src->attrs[srcix + i];
+                    caca_add_dirty_rect(dst, x + starti + i, y + j, 1, 1);
+                }
             }
         }
         else
         {
-            memcpy(dst->chars + dstix, src->chars + srcix, stride * 4);
-            memcpy(dst->attrs + dstix, src->attrs + srcix, stride * 4);
+            if(memcmp(dst->chars + dstix, src->chars + srcix, stride * 4) ||
+               memcmp(dst->attrs + dstix, src->attrs + srcix, stride * 4))
+            {
+                /* FIXME be more precise ? */
+                memcpy(dst->chars + dstix, src->chars + srcix, stride * 4);
+                memcpy(dst->attrs + dstix, src->attrs + srcix, stride * 4);
+                caca_add_dirty_rect(dst, x + starti, y + j, stride, 1);
+            }
         }
 
@@ -467 +478 @@
     }
 
-    caca_add_dirty_rect(dst, starti + x - bleed_left, startj + y,
-                        endi - starti + bleed_left + bleed_right,
-                        endj - startj);
 
     return 0;

libcaca/trunk/tests/dirty.cpp

@@ -29 +29 @@
     CPPUNIT_TEST(test_put_char_not_dirty);
     CPPUNIT_TEST(test_box);
+    CPPUNIT_TEST(test_blit);
     CPPUNIT_TEST_SUITE_END();
 
@@ -186 +187 @@
     }
 
+    void test_blit()
+    {
+        caca_canvas_t *cv, *cv2;
+        int i, dx, dy, dw, dh;
+
+        cv = caca_create_canvas(WIDTH, HEIGHT);
+        caca_clear_dirty_rect_list(cv);
+        cv2 = caca_create_canvas(2, 2);
+        caca_fill_box(cv2, 0, 0, 2, 1, 'x');
+
+        caca_blit(cv, 1, 1, cv2, NULL);
+        i = caca_get_dirty_rect_count(cv);
+        CPPUNIT_ASSERT_EQUAL(i, 1);
+        caca_get_dirty_rect(cv, 0, &dx, &dy, &dw, &dh);
+
+        /* Check that blitting a canvas make a dirty rectangle
+         * only for modified lines */
+        CPPUNIT_ASSERT(dx == 1);
+        CPPUNIT_ASSERT(dy == 1);
+        CPPUNIT_ASSERT(dw >= 2);
+        CPPUNIT_ASSERT(dh == 1);
+    }
+
 private:
     static int const WIDTH = 80, HEIGHT = 50;