On Thu, Jul 30, 2015 at 10:47 PM, Bryce Harrington
<br...@osg.samsung.com> wrote:
> Mind re-posting the patch, with updated changelog accounting for
> dropping the redundant renderer destruction.
>
> I did a quick look through the other compositor codes in case they
> contained similar code, but didn't spot anything.  Did you check as
> well?
>
> Anyway, looks good.
>
> Reviewed-by: Bryce Harrington <br...@osg.samsung.com>

Hi Bryce,

well, I did but I looked once again at the code and noticed similar
problem in foo_backend_create() in compositor-drm.c,
compositor-fbdev.c and compositor-rpi.c files. Like in
compositor-wayland.c they execute
         compositor->renderer->destroy(compositor);
before
         weston_compositor_shutdown(compositor);

I'll prepare a new patch. This time it will be send according to the
guidelines that I found here
http://wiki.x.org/wiki/Development/Documentation/SubmittingPatches/

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

Reply via email to