From 930ecef9b583a7dca2d886ed893dcfc684772d9c Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Thu, 1 Jul 2021 10:25:35 +1000 Subject: Cleanup: outdated IPO references in comments --- source/blender/editors/space_sequencer/sequencer_edit.c | 4 ++-- source/blender/editors/transform/transform_convert_action.c | 13 +++++++------ source/blender/editors/util/ed_util.c | 7 +++++-- 3 files changed, 14 insertions(+), 10 deletions(-) (limited to 'source/blender/editors') diff --git a/source/blender/editors/space_sequencer/sequencer_edit.c b/source/blender/editors/space_sequencer/sequencer_edit.c index e17beb228de..25009532dd0 100644 --- a/source/blender/editors/space_sequencer/sequencer_edit.c +++ b/source/blender/editors/space_sequencer/sequencer_edit.c @@ -1816,8 +1816,8 @@ static int sequencer_separate_images_exec(bContext *C, wmOperator *op) /* Remove seq so overlap tests don't conflict, * see seq_free_sequence below for the real freeing. */ BLI_remlink(ed->seqbasep, seq); - /* if (seq->ipo) id_us_min(&seq->ipo->id); */ - /* XXX, remove fcurve and assign to split image strips */ + /* TODO: remove f-curve and assign to split image strips. + * The old animation system would remove the user of `seq->ipo`. */ start_ofs = timeline_frame = SEQ_transform_get_left_handle_frame(seq); frame_end = SEQ_transform_get_right_handle_frame(seq); diff --git a/source/blender/editors/transform/transform_convert_action.c b/source/blender/editors/transform/transform_convert_action.c index deb64ca2892..cfa14e21d0d 100644 --- a/source/blender/editors/transform/transform_convert_action.c +++ b/source/blender/editors/transform/transform_convert_action.c @@ -210,12 +210,13 @@ static TransData *ActionFCurveToTransData(TransData *td, return td; } -/* This function advances the address to which td points to, so it must return +/** + * This function advances the address to which td points to, so it must return * the new address so that the next time new transform data is added, it doesn't - * overwrite the existing ones... i.e. td = GPLayerToTransData(td, ipo, ob, side, cfra); + * overwrite the existing ones: e.g. `td += GPLayerToTransData(td, ...);` * - * The 'side' argument is needed for the extend mode. 'B' = both sides, 'R'/'L' mean only data - * on the named side are used. + * \param side: is needed for the extend mode. 'B' = both sides, + * 'R'/'L' mean only data on the named side are used. */ static int GPLayerToTransData(TransData *td, tGPFtransdata *tfd, @@ -242,7 +243,7 @@ static int GPLayerToTransData(TransData *td, tfd->val = (float)gpf->framenum; tfd->sdata = &gpf->framenum; - /* advance td now */ + /* Advance `td` now. */ td++; tfd++; count++; @@ -339,7 +340,7 @@ void createTransActionData(bContext *C, TransInfo *t) t->frame_side = 'B'; } - /* loop 1: fully select ipo-keys and count how many BezTriples are selected */ + /* loop 1: fully select F-curve keys and count how many BezTriples are selected */ for (ale = anim_data.first; ale; ale = ale->next) { AnimData *adt = ANIM_nla_mapping_get(&ac, ale); int adt_count = 0; diff --git a/source/blender/editors/util/ed_util.c b/source/blender/editors/util/ed_util.c index b80782b51be..7bbdc58474f 100644 --- a/source/blender/editors/util/ed_util.c +++ b/source/blender/editors/util/ed_util.c @@ -295,8 +295,11 @@ bool ED_editors_flush_edits(Main *bmain) /* ***** XXX: functions are using old blender names, cleanup later ***** */ -/* now only used in 2d spaces, like time, ipo, nla, sima... */ -/* XXX shift/ctrl not configurable */ +/** + * Now only used in 2D spaces, like time, f-curve, NLA, image, etc. + * + * \note Shift/Control are not configurable key-bindings. + */ void apply_keyb_grid( int shift, int ctrl, float *val, float fac1, float fac2, float fac3, int invert) { -- cgit v1.2.3