On Sun, Jun 28, 2015 at 8:49 PM, Peter Hutterer <peter.hutte...@who-t.net>
wrote:

>
> +       if (elm->next == NULL && elm->prev == NULL)
> +               return;
> +
>         elm->prev->next = elm->next;
>         elm->next->prev = elm->prev;
>         elm->next = NULL;
>

 You probably don't need to check both pointers, as the code will crash if
only one of them is NULL.
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to