On Wed, 3 Jan 2018 16:09:21 +0100
Emre Ucan <eu...@de.adit-jv.com> wrote:

> it has only developermode option parameter.
> The parameter is only used in init_ivi_shell.
> Therefore, we can basically remove the struct,
> and check the option locally in the function.
> 
> Signed-off-by: Emre Ucan <eu...@de.adit-jv.com>
> ---
>  ivi-shell/ivi-shell.c | 42 ++++++++++++------------------------------
>  1 file changed, 12 insertions(+), 30 deletions(-)
> 
> diff --git a/ivi-shell/ivi-shell.c b/ivi-shell/ivi-shell.c
> index ebca2fc..51e13a0 100644
> --- a/ivi-shell/ivi-shell.c
> +++ b/ivi-shell/ivi-shell.c
> @@ -67,11 +67,6 @@ struct ivi_shell_surface
>       struct wl_list link;
>  };
>  
> -struct ivi_shell_setting
> -{
> -     int developermode;
> -};


Nice!

Reviewed-by: Pekka Paalanen <pekka.paala...@collabora.co.uk>


Thanks,
pq

Attachment: pgpB6ArzLLJCm.pgp
Description: OpenPGP digital signature

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

Reply via email to