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, &current_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, &current_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