[PATCH weston] weston-launch: Remove unused event source

Bryce Harrington bryce at osg.samsung.com
Fri Oct 23 12:35:28 PDT 2015


On Wed, Oct 21, 2015 at 12:39:11PM -0500, Derek Foreman wrote:
> vt_source is never added to an event loop in this launcher, but
> it's potentially removed.
> 
> Let's just remove the variable entirely.
> 
> Signed-off-by: Derek Foreman <derekf at osg.samsung.com>

Yep, nothing else appears to reference vt_source in this file.

Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>

To ssh://git.freedesktop.org/git/wayland/weston
   1ab9ca7..fa41eec  master -> master


> ---
>  src/launcher-weston-launch.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/src/launcher-weston-launch.c b/src/launcher-weston-launch.c
> index e85b2c9..ad919f1 100644
> --- a/src/launcher-weston-launch.c
> +++ b/src/launcher-weston-launch.c
> @@ -96,7 +96,6 @@ struct launcher_weston_launch {
>  	struct wl_event_source *source;
>  
>  	int kb_mode, tty, drm_fd;
> -	struct wl_event_source *vt_source;
>  };
>  
>  static int
> @@ -283,7 +282,6 @@ launcher_weston_launch_destroy(struct weston_launcher *launcher_base)
>  		wl_event_source_remove(launcher->source);
>  	} else {
>  		launcher_weston_launch_restore(&launcher->base);
> -		wl_event_source_remove(launcher->vt_source);
>  	}
>  
>  	if (launcher->tty >= 0)
> -- 
> 2.6.1
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list