[PATCH weston 06/21] compositor-wayland: move output init into common, fix error path

[Ray, Ian (GE Healthcare)]

On 22/09/2017, 17.33, "wayland-devel on behalf of Pekka Paalanen" <wayland-devel-bounces at lists.freedesktop.org on behalf of ppaalanen at gmail.com> wrote:
> 
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Move the weston_output_init() call into wayland_output_create_common().
> This avoids passing the name twice to different functions, and follows
> the precedent set in "libweston: weston_output_init(..., +name)" for
> calling init before accessing fields.
> 
> Since the error paths in wayland_output_create_for_parent_output() and
> wayland_output_create_fullscreen() are now guaranteed to have
> weston_output init'd, call weston_output_destroy() appropriately. There
> might be more to free than just the name.
> 
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

Reviewed-by: Ian Ray <ian.ray at ge.com>