From c36d2a9a7a2a01fdaeea98270ef1eb2314dfada3 Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Wed, 30 Jun 2021 14:35:01 +1000 Subject: Cleanup: rename 'count' to 'len' Reserve the term count for values that require calculation (typically linked lists). Without this convention it's difficult to know if using a length accessor function in a loop will be O(N^2) without inspecting the underlying implementation. --- source/blender/editors/transform/transform_snap_sequencer.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'source/blender/editors/transform') diff --git a/source/blender/editors/transform/transform_snap_sequencer.c b/source/blender/editors/transform/transform_snap_sequencer.c index 952b580a0a3..a180db349ff 100644 --- a/source/blender/editors/transform/transform_snap_sequencer.c +++ b/source/blender/editors/transform/transform_snap_sequencer.c @@ -52,14 +52,14 @@ typedef struct TransSeqSnapData { /** \name Snap sources * \{ */ -static int seq_get_snap_source_points_count(SeqCollection *snap_sources) +static int seq_get_snap_source_points_len(SeqCollection *snap_sources) { - return SEQ_collection_count(snap_sources) * 2; + return SEQ_collection_len(snap_sources) * 2; } static void seq_snap_source_points_alloc(TransSeqSnapData *snap_data, SeqCollection *snap_sources) { - const size_t point_count = seq_get_snap_source_points_count(snap_sources); + const size_t point_count = seq_get_snap_source_points_len(snap_sources); snap_data->source_snap_points = MEM_callocN(sizeof(int) * point_count, __func__); memset(snap_data->source_snap_points, 0, sizeof(int)); snap_data->source_snap_point_count = point_count; @@ -136,7 +136,7 @@ static int seq_get_snap_target_points_count(const TransInfo *t, count += 2; } - count *= SEQ_collection_count(snap_targets); + count *= SEQ_collection_len(snap_targets); if (snap_mode & SEQ_SNAP_TO_PLAYHEAD) { count++; -- cgit v1.2.3