[Libvisual-devel] list_destroy only destroys half the list

[Jonathan M. <jon...@ka...>]

Hi,

I noticed the list_destroy function in libvisual 0.4.0 only destroys
half of the list elements.

In this loop:

                while ((elem = visual_list_next (list, &le)) != NULL) {
                        destroyer (elem);
                        visual_list_delete (list, &le);
                }

both visual_list_next and visual_list_delete move 'le' to the next list 
element.  The attached patch makes valgrind much happier.

enjoy,
-jonathan

(not subscribed; please cc me on responses)