On Tue, 6 Feb 2018 13:53:21 +0000
Daniel Stone <dan...@fooishbar.org> wrote:

> Hi Pekka,
> 
> On 6 February 2018 at 13:30, Pekka Paalanen <ppaala...@gmail.com> wrote:
> > On Mon,  5 Feb 2018 18:44:15 +0000
> > Daniel Stone <dani...@collabora.com> wrote:  
> >> @@ -4296,7 +4290,8 @@ drm_output_destroy(struct weston_output *base)
> >>                */
> >>               if (output->cursor_plane)
> >>                       drm_plane_destroy(output->cursor_plane);
> >> -             drm_plane_destroy(output->scanout_plane);
> >> +             if (output->scanout_plane)
> >> +                     drm_plane_destroy(output->scanout_plane);  
> >
> > This hunk belong in the previous patch to make it correct.  
> 
> Yes, fixup landed in the wrong patch. Thanks for catching that. Is
> 'otherwise looks ok' on the previous patch R-b with this hunk moved
> back?

Yes, it is!


Thanks,
pq

Attachment: pgpqvvK53rmQO.pgp
Description: OpenPGP digital signature

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

Reply via email to