[PATCH] screenshooter: Add missing field initializers for wl_output_listener

[Jasper St. Pierre]

Shouldn't missing fields in structs be auto-initialized to 0 / NULL? I
thought that was part of the C specification.

On Tue, Jul 7, 2015 at 8:52 AM, Christopher Michael
<cpmichael at osg.samsung.com> wrote:
> This patch adds missing placeholders for the wl_output listener
> functions 'done' and 'scale. Currently these placeholders are being
> set to NULL as the done and scale callbacks are not used in the
> screenshot client.
>
> Signed-off-by: Chris Michael <cp.michael at samsung.com>
> ---
>  clients/screenshot.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/clients/screenshot.c b/clients/screenshot.c
> index f11e3ba..0d9b320 100644
> --- a/clients/screenshot.c
> +++ b/clients/screenshot.c
> @@ -114,7 +114,9 @@ display_handle_mode(void *data,
>
>  static const struct wl_output_listener output_listener = {
>      display_handle_geometry,
> -    display_handle_mode
> +    display_handle_mode,
> +    NULL,
> +    NULL,
>  };
>
>  static void
> --
> 2.4.4
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel



-- 
  Jasper