Cleanup: add SEQ_ALL_BEGIN, SEQ_CURRENT_BEGIN to .clang-format

These were missed in 70500121b457d which caused reformatting.
This commit is contained in:
Campbell Barton 2020-08-26 09:59:15 +10:00
parent be4abb42c1
commit 88ada5e218
7 changed files with 35 additions and 68 deletions

@ -253,8 +253,8 @@ ForEachMacros:
- RNA_STRUCT_BEGIN_SKIP_RNA_TYPE - RNA_STRUCT_BEGIN_SKIP_RNA_TYPE
- SCULPT_VERTEX_DUPLICATES_AND_NEIGHBORS_ITER_BEGIN - SCULPT_VERTEX_DUPLICATES_AND_NEIGHBORS_ITER_BEGIN
- SCULPT_VERTEX_NEIGHBORS_ITER_BEGIN - SCULPT_VERTEX_NEIGHBORS_ITER_BEGIN
- SEQP_BEGIN - SEQ_ALL_BEGIN
- SEQ_BEGIN - SEQ_CURRENT_BEGIN
- SURFACE_QUAD_ITER_BEGIN - SURFACE_QUAD_ITER_BEGIN
- foreach - foreach
- ED_screen_areas_iter - ED_screen_areas_iter

@ -507,8 +507,7 @@ void BKE_sequencer_editing_free(Scene *scene, const bool do_id_user)
BKE_sequencer_prefetch_free(scene); BKE_sequencer_prefetch_free(scene);
BKE_sequencer_cache_destruct(scene); BKE_sequencer_cache_destruct(scene);
SEQ_ALL_BEGIN(ed, seq) SEQ_ALL_BEGIN (ed, seq) {
{
/* handle cache freeing above */ /* handle cache freeing above */
BKE_sequence_free_ex(scene, seq, false, do_id_user, false); BKE_sequence_free_ex(scene, seq, false, do_id_user, false);
} }
@ -6172,8 +6171,7 @@ void BKE_sequencer_check_uuids_unique_and_report(const Scene *scene)
BLI_session_uuid_ghash_hash, BLI_session_uuid_ghash_compare, "sequencer used uuids"); BLI_session_uuid_ghash_hash, BLI_session_uuid_ghash_compare, "sequencer used uuids");
const Sequence *sequence; const Sequence *sequence;
SEQ_ALL_BEGIN(scene->ed, sequence) SEQ_ALL_BEGIN (scene->ed, sequence) {
{
const SessionUUID *session_uuid = &sequence->runtime.session_uuid; const SessionUUID *session_uuid = &sequence->runtime.session_uuid;
if (!BLI_session_uuid_is_generated(session_uuid)) { if (!BLI_session_uuid_is_generated(session_uuid)) {
printf("Sequence %s does not have UUID generated.\n", sequence->name); printf("Sequence %s does not have UUID generated.\n", sequence->name);

@ -1251,8 +1251,7 @@ void blo_do_versions_pre250(FileData *fd, Library *lib, Main *bmain)
while (sce) { while (sce) {
ed = sce->ed; ed = sce->ed;
if (ed) { if (ed) {
SEQ_ALL_BEGIN(sce->ed, seq) SEQ_ALL_BEGIN (sce->ed, seq) {
{
if (seq->type == SEQ_TYPE_IMAGE || seq->type == SEQ_TYPE_MOVIE) { if (seq->type == SEQ_TYPE_IMAGE || seq->type == SEQ_TYPE_MOVIE) {
seq->alpha_mode = SEQ_ALPHA_STRAIGHT; seq->alpha_mode = SEQ_ALPHA_STRAIGHT;
} }
@ -2443,8 +2442,7 @@ void blo_do_versions_pre250(FileData *fd, Library *lib, Main *bmain)
Sequence *seq; Sequence *seq;
for (sce = bmain->scenes.first; sce; sce = sce->id.next) { for (sce = bmain->scenes.first; sce; sce = sce->id.next) {
SEQ_ALL_BEGIN(sce->ed, seq) SEQ_ALL_BEGIN (sce->ed, seq) {
{
if (seq->blend_mode == 0) { if (seq->blend_mode == 0) {
seq->blend_opacity = 100.0f; seq->blend_opacity = 100.0f;
} }
@ -2597,8 +2595,7 @@ void blo_do_versions_pre250(FileData *fd, Library *lib, Main *bmain)
while (sce) { while (sce) {
ed = sce->ed; ed = sce->ed;
if (ed) { if (ed) {
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (seq->strip && seq->strip->proxy) { if (seq->strip && seq->strip->proxy) {
seq->strip->proxy->quality = 90; seq->strip->proxy->quality = 90;
} }

@ -217,8 +217,7 @@ static void seq_proxy_build_job(const bContext *C, ReportList *reports)
file_list = BLI_gset_new(BLI_ghashutil_strhash_p, BLI_ghashutil_strcmp, "file list"); file_list = BLI_gset_new(BLI_ghashutil_strhash_p, BLI_ghashutil_strcmp, "file list");
bool selected = false; /* Check for no selected strips */ bool selected = false; /* Check for no selected strips */
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (!ELEM(seq->type, SEQ_TYPE_MOVIE, SEQ_TYPE_IMAGE, SEQ_TYPE_META) || if (!ELEM(seq->type, SEQ_TYPE_MOVIE, SEQ_TYPE_IMAGE, SEQ_TYPE_META) ||
(seq->flag & SELECT) == 0) { (seq->flag & SELECT) == 0) {
continue; continue;
@ -484,8 +483,7 @@ void ED_sequencer_deselect_all(Scene *scene)
return; return;
} }
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
seq->flag &= ~SEQ_ALLSEL; seq->flag &= ~SEQ_ALLSEL;
} }
SEQ_CURRENT_END; SEQ_CURRENT_END;
@ -1068,8 +1066,7 @@ static void UNUSED_FUNCTION(seq_remap_paths)(Scene *scene)
return; return;
} }
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (seq->flag & SELECT) { if (seq->flag & SELECT) {
if (STREQLEN(seq->strip->dir, from, strlen(from))) { if (STREQLEN(seq->strip->dir, from, strlen(from))) {
printf("found %s\n", seq->strip->dir); printf("found %s\n", seq->strip->dir);
@ -2298,16 +2295,14 @@ static int sequencer_split_exec(bContext *C, wmOperator *op)
Sequence *seq; Sequence *seq;
if (ignore_selection) { if (ignore_selection) {
if (use_cursor_position) { if (use_cursor_position) {
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (seq->enddisp == split_frame && seq->machine == split_channel) { if (seq->enddisp == split_frame && seq->machine == split_channel) {
seq_selected = seq->flag & SEQ_ALLSEL; seq_selected = seq->flag & SEQ_ALLSEL;
} }
} }
SEQ_CURRENT_END; SEQ_CURRENT_END;
if (!seq_selected) { if (!seq_selected) {
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (seq->startdisp == split_frame && seq->machine == split_channel) { if (seq->startdisp == split_frame && seq->machine == split_channel) {
seq->flag &= ~SEQ_ALLSEL; seq->flag &= ~SEQ_ALLSEL;
} }
@ -2318,8 +2313,7 @@ static int sequencer_split_exec(bContext *C, wmOperator *op)
} }
else { else {
if (split_side != SEQ_SIDE_BOTH) { if (split_side != SEQ_SIDE_BOTH) {
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (split_side == SEQ_SIDE_LEFT) { if (split_side == SEQ_SIDE_LEFT) {
if (seq->startdisp >= split_frame) { if (seq->startdisp >= split_frame) {
seq->flag &= ~SEQ_ALLSEL; seq->flag &= ~SEQ_ALLSEL;
@ -2334,8 +2328,7 @@ static int sequencer_split_exec(bContext *C, wmOperator *op)
SEQ_CURRENT_END; SEQ_CURRENT_END;
} }
} }
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (seq->seq1 || seq->seq2 || seq->seq3) { if (seq->seq1 || seq->seq2 || seq->seq3) {
BKE_sequence_calc(scene, seq); BKE_sequence_calc(scene, seq);
} }
@ -2552,8 +2545,7 @@ static int sequencer_delete_exec(bContext *C, wmOperator *UNUSED(op))
BKE_sequencer_prefetch_stop(scene); BKE_sequencer_prefetch_stop(scene);
SEQ_CURRENT_BEGIN(scene->ed, seq) SEQ_CURRENT_BEGIN (scene->ed, seq) {
{
if (seq->flag & SELECT) { if (seq->flag & SELECT) {
BKE_sequencer_flag_for_removal(scene, ed->seqbasep, seq); BKE_sequencer_flag_for_removal(scene, ed->seqbasep, seq);
} }
@ -3508,8 +3500,7 @@ static int sequencer_rebuild_proxy_exec(bContext *C, wmOperator *UNUSED(op))
file_list = BLI_gset_new(BLI_ghashutil_strhash_p, BLI_ghashutil_strcmp, "file list"); file_list = BLI_gset_new(BLI_ghashutil_strhash_p, BLI_ghashutil_strcmp, "file list");
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if ((seq->flag & SELECT)) { if ((seq->flag & SELECT)) {
ListBase queue = {NULL, NULL}; ListBase queue = {NULL, NULL};
LinkData *link; LinkData *link;
@ -3577,8 +3568,7 @@ static int sequencer_enable_proxies_exec(bContext *C, wmOperator *op)
turnon = false; turnon = false;
} }
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if ((seq->flag & SELECT)) { if ((seq->flag & SELECT)) {
if (ELEM(seq->type, SEQ_TYPE_MOVIE, SEQ_TYPE_IMAGE, SEQ_TYPE_META)) { if (ELEM(seq->type, SEQ_TYPE_MOVIE, SEQ_TYPE_IMAGE, SEQ_TYPE_META)) {
BKE_sequencer_proxy_set(seq, turnon); BKE_sequencer_proxy_set(seq, turnon);
@ -4000,8 +3990,7 @@ static int sequencer_export_subtitles_exec(bContext *C, wmOperator *op)
return OPERATOR_CANCELLED; return OPERATOR_CANCELLED;
} }
SEQ_ALL_BEGIN(ed, seq) SEQ_ALL_BEGIN (ed, seq) {
{
if (seq->type == SEQ_TYPE_TEXT) { if (seq->type == SEQ_TYPE_TEXT) {
BLI_addtail(&text_seq, MEM_dupallocN(seq)); BLI_addtail(&text_seq, MEM_dupallocN(seq));
} }

@ -232,8 +232,7 @@ static int strip_modifier_copy_exec(bContext *C, wmOperator *op)
return OPERATOR_CANCELLED; return OPERATOR_CANCELLED;
} }
SEQ_CURRENT_BEGIN(ed, seq_iter) SEQ_CURRENT_BEGIN (ed, seq_iter) {
{
if (seq_iter->flag & SELECT) { if (seq_iter->flag & SELECT) {
if (seq_iter == seq) { if (seq_iter == seq) {
continue; continue;

@ -415,8 +415,7 @@ static int sequencer_select_exec(bContext *C, wmOperator *op)
const float x = UI_view2d_region_to_view_x(v2d, mval[0]); const float x = UI_view2d_region_to_view_x(v2d, mval[0]);
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (((x < CFRA) && (seq->enddisp <= CFRA)) || ((x >= CFRA) && (seq->startdisp >= CFRA))) { if (((x < CFRA) && (seq->enddisp <= CFRA)) || ((x >= CFRA) && (seq->startdisp >= CFRA))) {
/* Select left or right. */ /* Select left or right. */
seq->flag |= SELECT; seq->flag |= SELECT;
@ -976,8 +975,7 @@ static int sequencer_select_side_of_frame_exec(bContext *C, wmOperator *op)
ED_sequencer_deselect_all(scene); ED_sequencer_deselect_all(scene);
} }
const int cfra = CFRA; const int cfra = CFRA;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
bool test = false; bool test = false;
switch (side) { switch (side) {
case -1: case -1:
@ -1284,8 +1282,7 @@ static bool select_grouped_type(Editing *ed, Sequence *actseq, const int channel
Sequence *seq; Sequence *seq;
bool changed = false; bool changed = false;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == actseq->type) { if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == actseq->type) {
seq->flag |= SELECT; seq->flag |= SELECT;
changed = true; changed = true;
@ -1302,8 +1299,7 @@ static bool select_grouped_type_basic(Editing *ed, Sequence *actseq, const int c
bool changed = false; bool changed = false;
const bool is_sound = SEQ_IS_SOUND(actseq); const bool is_sound = SEQ_IS_SOUND(actseq);
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && (is_sound ? SEQ_IS_SOUND(seq) : !SEQ_IS_SOUND(seq))) { if (SEQ_CHANNEL_CHECK(seq, channel) && (is_sound ? SEQ_IS_SOUND(seq) : !SEQ_IS_SOUND(seq))) {
seq->flag |= SELECT; seq->flag |= SELECT;
changed = true; changed = true;
@ -1320,8 +1316,7 @@ static bool select_grouped_type_effect(Editing *ed, Sequence *actseq, const int
bool changed = false; bool changed = false;
const bool is_effect = SEQ_IS_EFFECT(actseq); const bool is_effect = SEQ_IS_EFFECT(actseq);
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && if (SEQ_CHANNEL_CHECK(seq, channel) &&
(is_effect ? SEQ_IS_EFFECT(seq) : !SEQ_IS_EFFECT(seq))) { (is_effect ? SEQ_IS_EFFECT(seq) : !SEQ_IS_EFFECT(seq))) {
seq->flag |= SELECT; seq->flag |= SELECT;
@ -1344,8 +1339,7 @@ static bool select_grouped_data(Editing *ed, Sequence *actseq, const int channel
} }
if (SEQ_HAS_PATH(actseq) && dir) { if (SEQ_HAS_PATH(actseq) && dir) {
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && SEQ_HAS_PATH(seq) && seq->strip && if (SEQ_CHANNEL_CHECK(seq, channel) && SEQ_HAS_PATH(seq) && seq->strip &&
STREQ(seq->strip->dir, dir)) { STREQ(seq->strip->dir, dir)) {
seq->flag |= SELECT; seq->flag |= SELECT;
@ -1356,8 +1350,7 @@ static bool select_grouped_data(Editing *ed, Sequence *actseq, const int channel
} }
else if (actseq->type == SEQ_TYPE_SCENE) { else if (actseq->type == SEQ_TYPE_SCENE) {
Scene *sce = actseq->scene; Scene *sce = actseq->scene;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == SEQ_TYPE_SCENE && seq->scene == sce) { if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == SEQ_TYPE_SCENE && seq->scene == sce) {
seq->flag |= SELECT; seq->flag |= SELECT;
changed = true; changed = true;
@ -1367,8 +1360,7 @@ static bool select_grouped_data(Editing *ed, Sequence *actseq, const int channel
} }
else if (actseq->type == SEQ_TYPE_MOVIECLIP) { else if (actseq->type == SEQ_TYPE_MOVIECLIP) {
MovieClip *clip = actseq->clip; MovieClip *clip = actseq->clip;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == SEQ_TYPE_MOVIECLIP && if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == SEQ_TYPE_MOVIECLIP &&
seq->clip == clip) { seq->clip == clip) {
seq->flag |= SELECT; seq->flag |= SELECT;
@ -1379,8 +1371,7 @@ static bool select_grouped_data(Editing *ed, Sequence *actseq, const int channel
} }
else if (actseq->type == SEQ_TYPE_MASK) { else if (actseq->type == SEQ_TYPE_MASK) {
struct Mask *mask = actseq->mask; struct Mask *mask = actseq->mask;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == SEQ_TYPE_MASK && seq->mask == mask) { if (SEQ_CHANNEL_CHECK(seq, channel) && seq->type == SEQ_TYPE_MASK && seq->mask == mask) {
seq->flag |= SELECT; seq->flag |= SELECT;
changed = true; changed = true;
@ -1403,8 +1394,7 @@ static bool select_grouped_effect(Editing *ed, Sequence *actseq, const int chann
effects[i] = false; effects[i] = false;
} }
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && (seq->type & SEQ_TYPE_EFFECT) && if (SEQ_CHANNEL_CHECK(seq, channel) && (seq->type & SEQ_TYPE_EFFECT) &&
ELEM(actseq, seq->seq1, seq->seq2, seq->seq3)) { ELEM(actseq, seq->seq1, seq->seq2, seq->seq3)) {
effects[seq->type] = true; effects[seq->type] = true;
@ -1412,8 +1402,7 @@ static bool select_grouped_effect(Editing *ed, Sequence *actseq, const int chann
} }
SEQ_CURRENT_END; SEQ_CURRENT_END;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (SEQ_CHANNEL_CHECK(seq, channel) && effects[seq->type]) { if (SEQ_CHANNEL_CHECK(seq, channel) && effects[seq->type]) {
if (seq->seq1) { if (seq->seq1) {
seq->seq1->flag |= SELECT; seq->seq1->flag |= SELECT;
@ -1437,8 +1426,7 @@ static bool select_grouped_time_overlap(Editing *ed, Sequence *actseq)
Sequence *seq; Sequence *seq;
bool changed = false; bool changed = false;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
if (seq->startdisp < actseq->enddisp && seq->enddisp > actseq->startdisp) { if (seq->startdisp < actseq->enddisp && seq->enddisp > actseq->startdisp) {
seq->flag |= SELECT; seq->flag |= SELECT;
changed = true; changed = true;
@ -1459,8 +1447,7 @@ static bool select_grouped_effect_link(Editing *ed, Sequence *actseq, const int
int machine = actseq->machine; int machine = actseq->machine;
SeqIterator iter; SeqIterator iter;
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
seq->tmp = NULL; seq->tmp = NULL;
} }
SEQ_CURRENT_END; SEQ_CURRENT_END;
@ -1536,8 +1523,7 @@ static int sequencer_select_grouped_exec(bContext *C, wmOperator *op)
bool changed = false; bool changed = false;
if (!extend) { if (!extend) {
SEQ_CURRENT_BEGIN(ed, seq) SEQ_CURRENT_BEGIN (ed, seq) {
{
seq->flag &= ~SELECT; seq->flag &= ~SELECT;
changed = true; changed = true;
} }

@ -623,8 +623,7 @@ static void rna_ColorManagedColorspaceSettings_reload_update(Main *bmain,
bool seq_found = false; bool seq_found = false;
if (&scene->sequencer_colorspace_settings != colorspace_settings) { if (&scene->sequencer_colorspace_settings != colorspace_settings) {
SEQ_ALL_BEGIN(scene->ed, seq) SEQ_ALL_BEGIN (scene->ed, seq) {
{
if (seq->strip && &seq->strip->colorspace_settings == colorspace_settings) { if (seq->strip && &seq->strip->colorspace_settings == colorspace_settings) {
seq_found = true; seq_found = true;
break; break;
@ -644,8 +643,7 @@ static void rna_ColorManagedColorspaceSettings_reload_update(Main *bmain,
BKE_sequence_invalidate_cache_preprocessed(scene, seq); BKE_sequence_invalidate_cache_preprocessed(scene, seq);
} }
else { else {
SEQ_ALL_BEGIN(scene->ed, seq) SEQ_ALL_BEGIN (scene->ed, seq) {
{
BKE_sequence_free_anim(seq); BKE_sequence_free_anim(seq);
} }
SEQ_ALL_END; SEQ_ALL_END;