[PATCH? weston] - wayland backend: outputs are destroyed twice

Dawid Gajownik gajownik at gmail.com
Thu Jul 30 19:26:40 PDT 2015


On Thu, Jul 30, 2015 at 10:47 PM, Bryce Harrington
<bryce at 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 <bryce at 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


More information about the wayland-devel mailing list