derekf pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=51d1360a182f6cc6d280a575c2ac7b06b6cf9cb8

commit 51d1360a182f6cc6d280a575c2ac7b06b6cf9cb8
Author: Derek Foreman <[email protected]>
Date:   Thu Jun 28 13:12:03 2018 -0500

    ecore_anim: Simplify timeline terminal condition
    
    Summary:
    We should only ever have a pos of 1.0 once, the current terminal
    condition gives the impression that it might be possible to have
    more than one 1.0 firing.
    
    This would break a lot of code.
    
    No functional change intended.
    Depends on D6464
    
    Reviewers: devilhorns, zmike
    
    Reviewed By: zmike
    
    Subscribers: cedric, #committers, zmike
    
    Tags: #efl
    
    Differential Revision: https://phab.enlightenment.org/D6465
---
 src/lib/ecore/ecore_anim.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/ecore/ecore_anim.c b/src/lib/ecore/ecore_anim.c
index 6f265d4cf0..6b7f946b83 100644
--- a/src/lib/ecore/ecore_anim.c
+++ b/src/lib/ecore/ecore_anim.c
@@ -989,7 +989,7 @@ _ecore_animator_run(void *data)
           pos = 0.0;
      }
    run_ret = animator->run_func(animator->run_data, pos);
-   if (t >= (animator->start + animator->run) && (pos >= 1.0)) run_ret = 
EINA_FALSE;
+   if (eina_dbl_exact(pos, 1.0)) run_ret = EINA_FALSE;
    return run_ret;
 }
 

-- 


Reply via email to