Almost complete UpdateResource implementation
Alexandre Julliard
julliard at winehq.org
Fri Sep 10 15:18:33 CDT 2004
Vincent Béron <vberon at mecano.gme.usherb.ca> writes:
> + if(!updates_list)
> + {
> + updates_list = current_updates;
> + list_init(&updates_list->entry);
> + }
> + else
> + list_add_after(&updates_list->entry, ¤t_updates->entry);
That's not the right way to use the list functions. updates_list
should be a separate struct list, that will make things a lot easier.
> + LIST_FOR_EACH(ptr, ¤t_updates->resources_list->entry)
> + {
> + current_resource = LIST_ENTRY(ptr, QUEUEDRESOURCE, entry);
> + free_queuedresource(current_resource);
> + }
You cannot free objects that are still on list, you have to remove
them first.
--
Alexandre Julliard
julliard at winehq.org
More information about the wine-devel
mailing list