From 439c9b0b8478336f987b532212650c59b5f9f30f Mon Sep 17 00:00:00 2001 From: Richard Antalik Date: Thu, 7 Oct 2021 00:35:57 +0200 Subject: Cleanup: VSE iterator semantics Use `BLI_gset_ensure_p_ex()` instead of `BLI_gset_insert()` after checking `BLI_gset_lookup()`. --- source/blender/sequencer/intern/iterator.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'source') diff --git a/source/blender/sequencer/intern/iterator.c b/source/blender/sequencer/intern/iterator.c index 2429405350b..e83d158881e 100644 --- a/source/blender/sequencer/intern/iterator.c +++ b/source/blender/sequencer/intern/iterator.c @@ -189,10 +189,12 @@ SeqCollection *SEQ_query_by_reference(Sequence *seq_reference, */ bool SEQ_collection_append_strip(Sequence *seq, SeqCollection *collection) { - if (BLI_gset_lookup(collection->set, seq) != NULL) { + void **key; + if (BLI_gset_ensure_p_ex(collection->set, seq, &key)) { return false; } - BLI_gset_insert(collection->set, seq); + + *key = (void *)seq; return true; } -- cgit v1.2.3