[PATCH] ivi-shell: Fix wrong condition to check return value of controller_module_init

Pekka Paalanen ppaalanen at gmail.com
Tue Dec 16 07:49:27 PST 2014


On Tue, 16 Dec 2014 19:37:22 +0900
Nobuhiko Tanibata <NOBUHIKO_TANIBATA at xddp.denso.co.jp> wrote:

> Signed-off-by: Nobuhiko Tanibata <NOBUHIKO_TANIBATA at xddp.denso.co.jp>
> ---
>  ivi-shell/ivi-layout.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c
> index a126483..6b2604e 100644
> --- a/ivi-shell/ivi-layout.c
> +++ b/ivi-shell/ivi-layout.c
> @@ -2954,9 +2954,9 @@ load_controller_modules(struct weston_compositor *compositor, const char *module
>  
>  		controller_module_init = weston_load_module(buffer, "controller_module_init");
>  		if (controller_module_init)
> -			if(!controller_module_init(compositor, argc, argv,
> +			if(controller_module_init(compositor, argc, argv,
>  					       &ivi_controller_interface,
> -					       sizeof(struct ivi_controller_interface))) {
> +					       sizeof(struct ivi_controller_interface)) != 0) {
>  				weston_log("ivi-shell: Initialization of controller module fails");
>  				return -1;
>  		}

Pushed, thanks,
pq


More information about the wayland-devel mailing list