From 8dfe2801ace57798f2df24a0c5b462a0b9c52443 Mon Sep 17 00:00:00 2001 From: Sergey Sharybin Date: Tue, 5 Nov 2019 14:31:24 +0100 Subject: Fix T71356: Motion Paths no longer update --- source/blender/editors/animation/anim_motion_paths.c | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) (limited to 'source') diff --git a/source/blender/editors/animation/anim_motion_paths.c b/source/blender/editors/animation/anim_motion_paths.c index e4eabf43493..e6ab3609507 100644 --- a/source/blender/editors/animation/anim_motion_paths.c +++ b/source/blender/editors/animation/anim_motion_paths.c @@ -236,10 +236,8 @@ static void motionpath_get_global_framerange(ListBase *targets, int *r_sfra, int static int motionpath_get_prev_keyframe(MPathTarget *mpt, DLRBT_Tree *fcu_keys, int current_frame) { - /* If the current frame is outside of the configured motion path range we ignore update of this - * motion path by using invalid frame range where start frame is above the end frame. */ if (current_frame <= mpt->mpath->start_frame) { - return INT_MAX; + return mpt->mpath->start_frame; } float current_frame_float = current_frame; @@ -262,10 +260,8 @@ static int motionpath_get_prev_prev_keyframe(MPathTarget *mpt, static int motionpath_get_next_keyframe(MPathTarget *mpt, DLRBT_Tree *fcu_keys, int current_frame) { - /* If the current frame is outside of the configured motion path range we ignore update of this - * motion path by using invalid frame range where start frame is above the end frame. */ if (current_frame >= mpt->mpath->end_frame) { - return INT_MIN; + return mpt->mpath->end_frame; } float current_frame_float = current_frame; @@ -305,6 +301,15 @@ static void motionpath_calculate_update_range(MPathTarget *mpt, int *r_sfra, int *r_efra) { + *r_sfra = INT_MAX; + *r_efra = INT_MIN; + + /* If the current frame is outside of the configured motion path range we ignore update of this + * motion path by using invalid frame range where start frame is above the end frame. */ + if (current_frame < mpt->mpath->start_frame || current_frame > mpt->mpath->end_frame) { + return; + } + /* Similar to the case when there is only a single keyframe: need to update en entire range to * a constant value. */ if (!motionpath_check_can_use_keyframe_range(mpt, adt, fcurve_list)) { @@ -313,9 +318,6 @@ static void motionpath_calculate_update_range(MPathTarget *mpt, return; } - *r_sfra = INT_MAX; - *r_efra = INT_MIN; - /* NOTE: Iterate over individual f-curves, and check their keyframes individually and pick a * widest range from them. This is because it's possible to have more narrow keyframe on a * channel which wasn't edited. -- cgit v1.2.3