Mercurial > MadButterfly

diff src/redraw_man.c @ 489:23c7667b3ec0 Android_Skia

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fix a potential bug when destroy a rdman. When a rdman is dirty, free shapes and coords works specially. Objects are append to a free list. They are not real freed until rdman being clean. redraw_man_destroy() free shapes and coords with free functions of them. If rdman is dirty when destroy it, objects would be leaked. The changeset make rdman clean before free shapes and coords to make objects being freed correctly.
author Thinker K.F. Li <thinker@branda.to>
date Sun, 22 Nov 2009 20:41:27 +0800
parents ba64f928542b
children 5d0b2761f89c
line wrap: on
line diff
--- a/src/redraw_man.c	Sun Nov 22 18:24:17 2009 +0800
+++ b/src/redraw_man.c	Sun Nov 22 20:41:27 2009 +0800
@@ -710,6 +710,12 @@
     free_free_objs(rdman);
     free_objs_destroy(rdman);
 
+    /* Mark rdman clean that shapes and coords can be freed
+     *  successfully.
+     */
+    DARRAY_CLEAN(&rdman->dirty_coords);
+    DARRAY_CLEAN(&rdman->dirty_geos);
+
     coord = postorder_coord_subtree(rdman->root_coord, NULL);
     while(coord) {
 	saved_coord = coord;