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