[PATCH weston] compositor: remove incorrect idle timer initialization

Pekka Paalanen ppaalanen at gmail.com
Thu Sep 24 03:55:18 PDT 2015


On Wed, 23 Sep 2015 18:29:46 +0300
Egor Starkov <egor.starkov at ge.com> wrote:

> weston_compositor_create() should just create idle timer but
> not arm it, because idle-time setting is not read at this point

Hi Egor,

the explanation in the cover letter is actually even better, so I
merged it with this message.

> Signed-off-by: Egor Starkov <egor.starkov at ge.com>
> ---
>  src/compositor.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/compositor.c b/src/compositor.c
> index cbef4dc..125afd5 100644
> --- a/src/compositor.c
> +++ b/src/compositor.c
> @@ -4527,7 +4527,6 @@ weston_compositor_create(struct wl_display *display, void *user_data)
>  
>  	loop = wl_display_get_event_loop(ec->wl_display);
>  	ec->idle_source = wl_event_loop_add_timer(loop, idle_handler, ec);
> -	wl_event_source_timer_update(ec->idle_source, ec->idle_time * 1000);
>  
>  	ec->input_loop = wl_event_loop_create();
>  

R-b me, and pushed:
   44bbdd0..56fe1c7  master -> master


Thanks,
pq
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20150924/3fa22367/attachment.sig>


More information about the wayland-devel mailing list