Hi Pekka,

On 16 February 2018 at 15:57, Pekka Paalanen <ppaala...@gmail.com> wrote:
> The head was just zalloc()'d, there is no need to memset it to zero.
>
> If a function fails, it is preferable it leaves the output arguments
> untouched.

Everything to this point is:
Reviewed-by: Daniel Stone <dani...@collabora.com>

and I'm happy for you to merge. Given I'm away today and tomorrow, I
won't be able to properly test multi-head and hotplugging until next
week, so if you want to go ahead and land it, I'll rebase the
remainder of the atomic series on top of that, fix the inevitable
bugs, and send it out.

I'm really happy with how this series has unfolded: there were a
couple of small places where things seemed mis-sequenced, but on the
whole it was very easy to read through and find a coherent story. Much
less scary than I feared. Thanks!

Cheers,
Daniel
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to