[Bf-blender-cvs] [12a8d7e] master: Cleanup: Make select grouped effect code a bit more readable

Antony Riakiotakis noreply at git.blender.org
Wed Jul 1 17:37:21 CEST 2015


Commit: 12a8d7e6c40c8f01699363d23190b556e6877520
Author: Antony Riakiotakis
Date:   Wed Jul 1 17:37:14 2015 +0200
Branches: master
https://developer.blender.org/rB12a8d7e6c40c8f01699363d23190b556e6877520

Cleanup: Make select grouped effect code a bit more readable

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

M	source/blender/editors/space_sequencer/sequencer_select.c
M	source/blender/makesdna/DNA_sequence_types.h

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

diff --git a/source/blender/editors/space_sequencer/sequencer_select.c b/source/blender/editors/space_sequencer/sequencer_select.c
index 4fbe0c6..c9fdb4a 100644
--- a/source/blender/editors/space_sequencer/sequencer_select.c
+++ b/source/blender/editors/space_sequencer/sequencer_select.c
@@ -1044,15 +1044,15 @@ static bool select_grouped_effect(Editing *ed, Sequence *actseq)
 {
 	Sequence *seq;
 	bool changed = false;
-	bool effects[SEQ_TYPE_EFFECT_MAX + 1];
+	bool effects[SEQ_TYPE_MAX + 1];
 	int i;
 
-	for (i = 0; i <= SEQ_TYPE_EFFECT_MAX; i++)
+	for (i = 0; i <= SEQ_TYPE_MAX; i++)
 		effects[i] = false;
 
 	SEQP_BEGIN (ed, seq)
 	{
-		if (ELEM(actseq, seq->seq1, seq->seq2, seq->seq3)) {
+		if ((seq->type & SEQ_TYPE_EFFECT) && ELEM(actseq, seq->seq1, seq->seq2, seq->seq3)) {
 			effects[seq->type] = true;
 		}
 	}
diff --git a/source/blender/makesdna/DNA_sequence_types.h b/source/blender/makesdna/DNA_sequence_types.h
index 76c27fd..7904d05 100644
--- a/source/blender/makesdna/DNA_sequence_types.h
+++ b/source/blender/makesdna/DNA_sequence_types.h
@@ -463,7 +463,7 @@ enum {
 	SEQ_TYPE_MULTICAM    = 30,
 	SEQ_TYPE_ADJUSTMENT  = 31,
 	SEQ_TYPE_GAUSSIAN_BLUR = 40,
-	SEQ_TYPE_EFFECT_MAX  = 40
+	SEQ_TYPE_MAX  = 40
 };
 
 #define SEQ_MOVIECLIP_RENDER_UNDISTORTED (1 << 0)




More information about the Bf-blender-cvs mailing list