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 <[email protected]>

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

Reviewed-by: Bryce Harrington <[email protected]>

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
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to