<div class="gmail_quote">On Thu, Jun 7, 2012 at 3:15 PM, Kristian Høgsberg <<a href="mailto:hoegsberg@gmail.com" target="_blank">hoegsberg@gmail.com</a>> wrote: <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div class="im">On Thu, Jun 07, 2012 at 09:12:30AM -0600, Scott Moreau wrote: > --- > src/compositor.c | 8 +++++++- > src/compositor.h | 2 ++ > 2 files changed, 9 insertions(+), 1 deletions(-) > > diff --git a/src/compositor.c b/src/compositor.c > index 9110d4c..7f51424 100644 > --- a/src/compositor.c > +++ b/src/compositor.c > @@ -980,6 +980,12 @@ fade_frame(struct weston_animation *animation, > } > } > > +WL_EXPORT int > +weston_animation_exists(struct weston_animation *animation) </div>Call it weston_animation_running() instead? </blockquote><div> Yes, I think this would be much better. </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <div><div class="h5"> > +{ > + return wl_list_empty(&animation->link) ? 0 : 1; > +} > + > WL_EXPORT void > weston_animation_run(struct weston_compositor *compositor, > struct weston_output *output, > @@ -1204,7 +1210,7 @@ weston_compositor_fade(struct weston_compositor *compositor, float tint) > } > > weston_surface_damage(compositor->fade.surface); > - if (wl_list_empty(&compositor->fade.animation.link)) > + if (!weston_animation_exists(&compositor->fade.animation)) > weston_animation_run(compositor, NULL, > &compositor->fade.animation); > } > diff --git a/src/compositor.h b/src/compositor.h > index 30cb7bb..69bbbc4 100644 > --- a/src/compositor.h > +++ b/src/compositor.h > @@ -534,6 +534,8 @@ weston_compositor_activity(struct weston_compositor *compositor); > void > weston_compositor_update_drag_surfaces(struct weston_compositor *compositor); > > +int > +weston_animation_exists(struct weston_animation *animation); > void > weston_animation_run(struct weston_compositor *compositor, > struct weston_output *output, > -- > 1.7.7.6 > </div></div>> _______________________________________________ > wayland-devel mailing list > <a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a> > <a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a> </blockquote></div>