diff options
author | Campbell Barton <ideasman42@gmail.com> | 2019-04-22 02:19:45 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2019-04-22 12:48:16 +0300 |
commit | 620b960d3d8cfd90b9f0df6ba3671c33eccb8309 (patch) | |
tree | 64f69db4bf9d44f0a32d1c92b0714bf2dc98ff2d /source/blender/editors/transform/transform_generics.c | |
parent | bba60bb564cf5a16cfcac744d4ba82cf8eba3da9 (diff) |
Cleanup: style, use braces for editors
Diffstat (limited to 'source/blender/editors/transform/transform_generics.c')
-rw-r--r-- | source/blender/editors/transform/transform_generics.c | 138 |
1 files changed, 92 insertions, 46 deletions
diff --git a/source/blender/editors/transform/transform_generics.c b/source/blender/editors/transform/transform_generics.c index 09593a44a54..e816f4b31ff 100644 --- a/source/blender/editors/transform/transform_generics.c +++ b/source/blender/editors/transform/transform_generics.c @@ -167,13 +167,16 @@ static void clipMirrorModifier(TransInfo *t) int clip; float loc[3], iloc[3]; - if (td->flag & TD_NOACTION) + if (td->flag & TD_NOACTION) { break; - if (td->loc == NULL) + } + if (td->loc == NULL) { break; + } - if (td->flag & TD_SKIP) + if (td->flag & TD_SKIP) { continue; + } copy_v3_v3(loc, td->loc); copy_v3_v3(iloc, td->iloc); @@ -227,12 +230,15 @@ static void editbmesh_apply_to_mirror(TransInfo *t) int i; for (i = 0; i < tc->data_len; i++, td++) { - if (td->flag & TD_NOACTION) + if (td->flag & TD_NOACTION) { break; - if (td->loc == NULL) + } + if (td->loc == NULL) { break; - if (td->flag & TD_SKIP) + } + if (td->flag & TD_SKIP) { continue; + } eve = td->extra; if (eve) { @@ -255,8 +261,9 @@ static void animrecord_check_state(Scene *scene, ID *id, wmTimer *animtimer) ScreenAnimData *sad = (animtimer) ? animtimer->customdata : NULL; /* sanity checks */ - if (ELEM(NULL, scene, id, sad)) + if (ELEM(NULL, scene, id, sad)) { return; + } /* check if we need a new strip if: * - if animtimer is running @@ -329,12 +336,14 @@ static bool fcu_test_selected(FCurve *fcu) BezTriple *bezt = fcu->bezt; unsigned int i; - if (bezt == NULL) /* ignore baked */ + if (bezt == NULL) { /* ignore baked */ return 0; + } for (i = 0; i < fcu->totvert; i++, bezt++) { - if (BEZT_ISSEL_ANY(bezt)) + if (BEZT_ISSEL_ANY(bezt)) { return 1; + } } return 0; @@ -430,25 +439,30 @@ static void recalcData_graphedit(TransInfo *t) FCurve *fcu = (FCurve *)ale->key_data; /* ignore FC-Curves without any selected verts */ - if (!fcu_test_selected(fcu)) + if (!fcu_test_selected(fcu)) { continue; + } /* watch it: if the time is wrong: do not correct handles yet */ - if (test_time_fcurve(fcu)) + if (test_time_fcurve(fcu)) { dosort++; - else + } + else { calchandles_fcurve(fcu); + } /* set refresh tags for objects using this animation, * BUT only if realtime updates are enabled */ - if ((sipo->flag & SIPO_NOREALTIMEUPDATES) == 0) + if ((sipo->flag & SIPO_NOREALTIMEUPDATES) == 0) { ANIM_list_elem_update(CTX_data_main(t->context), t->scene, ale); + } } /* do resort and other updates? */ - if (dosort) + if (dosort) { remake_graph_transdata(t, &anim_data); + } /* now free temp channels */ ANIM_animdata_freelist(&anim_data); @@ -476,14 +490,16 @@ static void recalcData_nla(TransInfo *t) int delta_y1, delta_y2; /* if this tdn has no handles, that means it is just a dummy that should be skipped */ - if (tdn->handle == 0) + if (tdn->handle == 0) { continue; + } /* set refresh tags for objects using this animation, * BUT only if realtime updates are enabled */ - if ((snla->flag & SNLA_NOREALTIMEUPDATES) == 0) + if ((snla->flag & SNLA_NOREALTIMEUPDATES) == 0) { ANIM_id_update(CTX_data_main(t->context), tdn->id); + } /* if canceling transform, just write the values without validating, then move on */ if (t->state == TRANS_CANCEL) { @@ -494,14 +510,16 @@ static void recalcData_nla(TransInfo *t) /* start */ strip->start = tdn->h1[0]; - if ((strip->prev) && (strip->prev->type == NLASTRIP_TYPE_TRANSITION)) + if ((strip->prev) && (strip->prev->type == NLASTRIP_TYPE_TRANSITION)) { strip->prev->end = tdn->h1[0]; + } /* end */ strip->end = tdn->h2[0]; - if ((strip->next) && (strip->next->type == NLASTRIP_TYPE_TRANSITION)) + if ((strip->next) && (strip->next->type == NLASTRIP_TYPE_TRANSITION)) { strip->next->start = tdn->h2[0]; + } /* flush transforms to child strips (since this should be a meta) */ BKE_nlameta_flush_transforms(strip); @@ -557,8 +575,9 @@ static void recalcData_nla(TransInfo *t) tdn->h1[0] += offset; tdn->h2[0] += offset; } - else /* all is fine and well */ + else { /* all is fine and well */ break; + } } /* handle auto-snapping @@ -649,8 +668,9 @@ static void recalcData_nla(TransInfo *t) tdn->nlt = track; tdn->trackIndex++; } - else /* can't move any further */ + else { /* can't move any further */ break; + } } } else { @@ -667,8 +687,9 @@ static void recalcData_nla(TransInfo *t) tdn->nlt = track; tdn->trackIndex--; } - else /* can't move any further */ + else { /* can't move any further */ break; + } } } } @@ -697,8 +718,9 @@ static void recalcData_image(TransInfo *t) SpaceImage *sima = t->sa->spacedata.first; flushTransUVs(t); - if (sima->flag & SI_LIVE_UNWRAP) + if (sima->flag & SI_LIVE_UNWRAP) { ED_uvedit_live_unwrap_re_solve(); + } FOREACH_TRANS_DATA_CONTAINER (t, tc) { if (tc->data_len) { @@ -727,20 +749,25 @@ static void recalcData_spaceclip(TransInfo *t) MovieTrackingMarker *marker = BKE_tracking_marker_get(track, framenr); if (t->mode == TFM_TRANSLATION) { - if (TRACK_AREA_SELECTED(track, TRACK_AREA_PAT)) + if (TRACK_AREA_SELECTED(track, TRACK_AREA_PAT)) { BKE_tracking_marker_clamp(marker, CLAMP_PAT_POS); - if (TRACK_AREA_SELECTED(track, TRACK_AREA_SEARCH)) + } + if (TRACK_AREA_SELECTED(track, TRACK_AREA_SEARCH)) { BKE_tracking_marker_clamp(marker, CLAMP_SEARCH_POS); + } } else if (t->mode == TFM_RESIZE) { - if (TRACK_AREA_SELECTED(track, TRACK_AREA_PAT)) + if (TRACK_AREA_SELECTED(track, TRACK_AREA_PAT)) { BKE_tracking_marker_clamp(marker, CLAMP_PAT_DIM); - if (TRACK_AREA_SELECTED(track, TRACK_AREA_SEARCH)) + } + if (TRACK_AREA_SELECTED(track, TRACK_AREA_SEARCH)) { BKE_tracking_marker_clamp(marker, CLAMP_SEARCH_DIM); + } } else if (t->mode == TFM_ROTATION) { - if (TRACK_AREA_SELECTED(track, TRACK_AREA_PAT)) + if (TRACK_AREA_SELECTED(track, TRACK_AREA_PAT)) { BKE_tracking_marker_clamp(marker, CLAMP_PAT_POS); + } } } @@ -851,14 +878,16 @@ static void recalcData_objects(TransInfo *t) /* If this bone has a parent tip that has been moved */ if (ebo_parent->flag & BONE_TIPSEL) { copy_v3_v3(ebo->head, ebo_parent->tail); - if (t->mode == TFM_BONE_ENVELOPE) + if (t->mode == TFM_BONE_ENVELOPE) { ebo->rad_head = ebo_parent->rad_tail; + } } /* If this bone has a parent tip that has NOT been moved */ else { copy_v3_v3(ebo_parent->tail, ebo->head); - if (t->mode == TFM_BONE_ENVELOPE) + if (t->mode == TFM_BONE_ENVELOPE) { ebo_parent->rad_tail = ebo->rad_head; + } } } @@ -869,8 +898,9 @@ static void recalcData_objects(TransInfo *t) ebo->rad_tail = 0.10f * ebo->length; ebo->dist = 0.25f * ebo->length; if (ebo->parent) { - if (ebo->rad_head > ebo->parent->rad_tail) + if (ebo->rad_head > ebo->parent->rad_tail) { ebo->rad_head = ebo->parent->rad_tail; + } } } else if (t->mode != TFM_BONE_ENVELOPE) { @@ -1005,11 +1035,13 @@ static void recalcData_objects(TransInfo *t) for (int i = 0; i < tc->data_len; i++, td++) { Object *ob = td->ob; - if (td->flag & TD_NOACTION) + if (td->flag & TD_NOACTION) { break; + } - if (td->flag & TD_SKIP) + if (td->flag & TD_SKIP) { continue; + } /* if animtimer is running, and the object already has animation data, * check if the auto-record feature means that we should record 'samples' @@ -1029,8 +1061,9 @@ static void recalcData_objects(TransInfo *t) */ DEG_id_tag_update(&ob->id, ID_RECALC_TRANSFORM); - if (t->flag & T_TEXTURE) + if (t->flag & T_TEXTURE) { DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY); + } } } @@ -1513,10 +1546,12 @@ void initTransInfo(bContext *C, TransInfo *t, wmOperator *op, const wmEvent *eve t->view = &ar->v2d; t->around = sclip->around; - if (ED_space_clip_check_show_trackedit(sclip)) + if (ED_space_clip_check_show_trackedit(sclip)) { t->options |= CTX_MOVIECLIP; - else if (ED_space_clip_check_show_maskedit(sclip)) + } + else if (ED_space_clip_check_show_maskedit(sclip)) { t->options |= CTX_MASK; + } } else { if (ar) { @@ -1724,14 +1759,18 @@ void freeTransCustomDataForMode(TransInfo *t) /* Here I would suggest only TransInfo related issues, like free data & reset vars. Not redraws */ void postTrans(bContext *C, TransInfo *t) { - if (t->draw_handle_view) + if (t->draw_handle_view) { ED_region_draw_cb_exit(t->ar->type, t->draw_handle_view); - if (t->draw_handle_apply) + } + if (t->draw_handle_apply) { ED_region_draw_cb_exit(t->ar->type, t->draw_handle_apply); - if (t->draw_handle_pixel) + } + if (t->draw_handle_pixel) { ED_region_draw_cb_exit(t->ar->type, t->draw_handle_pixel); - if (t->draw_handle_cursor) + } + if (t->draw_handle_cursor) { WM_paint_cursor_end(CTX_wm_manager(C), t->draw_handle_cursor); + } if (t->flag & T_MODAL_CURSOR_SET) { WM_cursor_modal_restore(CTX_wm_window(C)); @@ -1773,8 +1812,9 @@ void postTrans(bContext *C, TransInfo *t) } else { SpaceImage *sima = t->sa->spacedata.first; - if (sima->flag & SI_LIVE_UNWRAP) + if (sima->flag & SI_LIVE_UNWRAP) { ED_uvedit_live_unwrap_end(t->state == TRANS_CANCEL); + } } } else if (t->spacetype == SPACE_VIEW3D) { @@ -2085,12 +2125,15 @@ static void calculateCenter_FromAround(TransInfo *t, int around, float r_center[ calculateCenterMedian(t, r_center); break; case V3D_AROUND_CURSOR: - if (ELEM(t->spacetype, SPACE_IMAGE, SPACE_CLIP)) + if (ELEM(t->spacetype, SPACE_IMAGE, SPACE_CLIP)) { calculateCenterCursor2D(t, r_center); - else if (t->spacetype == SPACE_GRAPH) + } + else if (t->spacetype == SPACE_GRAPH) { calculateCenterCursorGraph2D(t, r_center); - else + } + else { calculateCenterCursor(t, r_center); + } break; case V3D_AROUND_LOCAL_ORIGINS: /* Individual element center uses median center for helpline and such */ @@ -2222,18 +2265,21 @@ void calculatePropRatio(TransInfo *t) /* Use rdist for falloff calculations, it is the real distance */ td->flag &= ~TD_NOACTION; - if (connected) + if (connected) { dist = (t->prop_size - td->dist) / t->prop_size; - else + } + else { dist = (t->prop_size - td->rdist) / t->prop_size; + } /* * Clamp to positive numbers. * Certain corner cases with connectivity and individual centers * can give values of rdist larger than propsize. */ - if (dist < 0.0f) + if (dist < 0.0f) { dist = 0.0f; + } switch (t->prop_mode) { case PROP_SHARP: |