On 15.12.2015 19:04, Thomas Klausner wrote:
> Like this?

[...]

> diff --git a/include/list.h b/include/list.h
> index 39f1985..f41d418 100644
> --- a/include/list.h
> +++ b/include/list.h
> @@ -317,6 +317,7 @@ xorg_list_is_empty(struct xorg_list *head)
>   * See xorg_list_for_each_entry for more details.
>   */
>  #define xorg_list_for_each_entry_safe(pos, tmp, head, member)                
> \
> +    pos = NULL;                                                              
> \
>      for (pos = __container_of((head)->next, pos, member),            \
>        tmp = __container_of(pos->member.next, pos, member);           \
>        &pos->member != (head);                                        \

Yes, but the xorg_list_for_each_entry macro needs the same treatment.
With that,

Acked-by: Michel Dänzer <michel.daen...@amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to