[Bf-blender-cvs] [2af789d1f3b] master: Added some TODO remarks.

Jeroen Bakker noreply at git.blender.org
Wed Aug 4 16:06:02 CEST 2021


Commit: 2af789d1f3b8a36d0578997bea5f907e31a613ed
Author: Jeroen Bakker
Date:   Wed Aug 4 16:05:49 2021 +0200
Branches: master
https://developer.blender.org/rB2af789d1f3b8a36d0578997bea5f907e31a613ed

Added some TODO remarks.

===================================================================

M	source/blender/editors/animation/anim_draw.c
M	source/blender/editors/animation/anim_motion_paths.c

===================================================================

diff --git a/source/blender/editors/animation/anim_draw.c b/source/blender/editors/animation/anim_draw.c
index 6469c47ab11..735f3b86924 100644
--- a/source/blender/editors/animation/anim_draw.c
+++ b/source/blender/editors/animation/anim_draw.c
@@ -522,6 +522,7 @@ static bool find_prev_next_keyframes(struct bContext *C, int *r_nextfra, int *r_
     mask_to_keylist(&ads, masklay, keylist);
   }
 
+  /* TODO(jbakker): Keylists are ordered, no need to do any searching at all. */
   /* find matching keyframe in the right direction */
   do {
     aknext = ED_keylist_find_next(keylist, cfranext);
diff --git a/source/blender/editors/animation/anim_motion_paths.c b/source/blender/editors/animation/anim_motion_paths.c
index 2a3ae35aab0..d976f5f72ad 100644
--- a/source/blender/editors/animation/anim_motion_paths.c
+++ b/source/blender/editors/animation/anim_motion_paths.c
@@ -234,6 +234,7 @@ static void motionpath_get_global_framerange(ListBase *targets, int *r_sfra, int
   }
 }
 
+/* TODO(jbakker): Remove complexity, keylists are ordered. */
 static int motionpath_get_prev_keyframe(MPathTarget *mpt,
                                         struct AnimKeylist *keylist,
                                         int current_frame)



More information about the Bf-blender-cvs mailing list