From 44db9f192eba5ef3c2f3117457b45540d3ca3801 Mon Sep 17 00:00:00 2001 From: Richard Antalik Date: Tue, 28 Dec 2021 14:28:52 +0100 Subject: Cleanup: Factor in VSE effect processing 2 factor variables were passed to effects, but they were hard coded to have same value. Remove duplicate variable from arguments, rename single argument to `fac`. Inverted factor variables were renamed to `mfac`. Any other factor related variables are prefixed with `temp_`. There should be no functional changes. --- source/blender/sequencer/SEQ_effects.h | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) (limited to 'source/blender/sequencer/SEQ_effects.h') diff --git a/source/blender/sequencer/SEQ_effects.h b/source/blender/sequencer/SEQ_effects.h index 4bd5b54b36b..87a8288a593 100644 --- a/source/blender/sequencer/SEQ_effects.h +++ b/source/blender/sequencer/SEQ_effects.h @@ -68,10 +68,10 @@ struct SeqEffectHandle { * 0: no early out, * 1: out = ibuf1, * 2: out = ibuf2 */ - int (*early_out)(struct Sequence *seq, float facf0, float facf1); + int (*early_out)(struct Sequence *seq, float fac); - /* stores the default facf0 and facf1 if no IPO is present */ - void (*get_default_fac)(struct Sequence *seq, float timeline_frame, float *facf0, float *facf1); + /* sets the default `fac` value */ + void (*get_default_fac)(struct Sequence *seq, float timeline_frame, float *fac); /* execute the effect * sequence effects are only required to either support @@ -81,8 +81,7 @@ struct SeqEffectHandle { struct ImBuf *(*execute)(const struct SeqRenderData *context, struct Sequence *seq, float timeline_frame, - float facf0, - float facf1, + float fac, struct ImBuf *ibuf1, struct ImBuf *ibuf2, struct ImBuf *ibuf3); @@ -95,8 +94,7 @@ struct SeqEffectHandle { void (*execute_slice)(const struct SeqRenderData *context, struct Sequence *seq, float timeline_frame, - float facf0, - float facf1, + float fac, struct ImBuf *ibuf1, struct ImBuf *ibuf2, struct ImBuf *ibuf3, -- cgit v1.2.3