[PATCH weston 6/6] ivi-shell: remove ivi_shell_setting

[Pekka Paalanen]

On Wed, 3 Jan 2018 16:09:21 +0100
Emre Ucan <eucan at 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 <eucan at 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.paalanen at collabora.co.uk>


Thanks,
pq

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20180122/3feed9e8/attachment.sig>