diff options
author | Sergey Sharybin <sergey@blender.org> | 2021-02-22 17:45:09 +0300 |
---|---|---|
committer | Sergey Sharybin <sergey@blender.org> | 2021-02-22 18:30:04 +0300 |
commit | dec73ed4099f98cd5f109d2bc036c07f48abc73b (patch) | |
tree | 3b0eeefc5715a19026fb7ea54322fc3665bfa9a3 /source/blender | |
parent | da5eed7ee6bc85104a484b67555457daf0ea369f (diff) |
Cleanup: Tracking, reduce indentation level
Diffstat (limited to 'source/blender')
-rw-r--r-- | source/blender/blenkernel/intern/movieclip.c | 102 |
1 files changed, 52 insertions, 50 deletions
diff --git a/source/blender/blenkernel/intern/movieclip.c b/source/blender/blenkernel/intern/movieclip.c index c1ebd06465b..85b1362c11e 100644 --- a/source/blender/blenkernel/intern/movieclip.c +++ b/source/blender/blenkernel/intern/movieclip.c @@ -1728,76 +1728,78 @@ void BKE_movieclip_update_scopes(MovieClip *clip, MovieClipUser *user, MovieClip scopes->track = NULL; scopes->track_locked = true; - if (clip) { - MovieTrackingTrack *act_track = BKE_tracking_track_get_active(&clip->tracking); - - if (act_track) { - MovieTrackingTrack *track = act_track; - int framenr = BKE_movieclip_remap_scene_to_clip_frame(clip, user->framenr); - MovieTrackingMarker *marker = BKE_tracking_marker_get(track, framenr); - - scopes->marker = marker; - scopes->track = track; - - if (marker->flag & MARKER_DISABLED) { - scopes->track_disabled = true; - } - else { - ImBuf *ibuf = BKE_movieclip_get_ibuf(clip, user); + scopes->framenr = user->framenr; + scopes->ok = true; - scopes->track_disabled = false; + if (clip == NULL) { + return; + } - if (ibuf && (ibuf->rect || ibuf->rect_float)) { - MovieTrackingMarker undist_marker = *marker; + MovieTrackingTrack *track = BKE_tracking_track_get_active(&clip->tracking); + if (track == NULL) { + return; + } - if (user->render_flag & MCLIP_PROXY_RENDER_UNDISTORT) { - int width, height; - float aspy = 1.0f / clip->tracking.camera.pixel_aspect; + const int framenr = BKE_movieclip_remap_scene_to_clip_frame(clip, user->framenr); + MovieTrackingMarker *marker = BKE_tracking_marker_get(track, framenr); - BKE_movieclip_get_size(clip, user, &width, &height); + scopes->marker = marker; + scopes->track = track; - undist_marker.pos[0] *= width; - undist_marker.pos[1] *= height * aspy; + if (marker->flag & MARKER_DISABLED) { + scopes->track_disabled = true; + } + else { + ImBuf *ibuf = BKE_movieclip_get_ibuf(clip, user); - BKE_tracking_undistort_v2( - &clip->tracking, width, height, undist_marker.pos, undist_marker.pos); + scopes->track_disabled = false; - undist_marker.pos[0] /= width; - undist_marker.pos[1] /= height * aspy; - } + if (ibuf && (ibuf->rect || ibuf->rect_float)) { + MovieTrackingMarker undist_marker = *marker; - scopes->track_search = BKE_tracking_get_search_imbuf( - ibuf, track, &undist_marker, true, true); + if (user->render_flag & MCLIP_PROXY_RENDER_UNDISTORT) { + int width, height; + float aspy = 1.0f / clip->tracking.camera.pixel_aspect; - scopes->undist_marker = undist_marker; + BKE_movieclip_get_size(clip, user, &width, &height); - scopes->frame_width = ibuf->x; - scopes->frame_height = ibuf->y; + undist_marker.pos[0] *= width; + undist_marker.pos[1] *= height * aspy; - scopes->use_track_mask = (track->flag & TRACK_PREVIEW_ALPHA) != 0; - } + BKE_tracking_undistort_v2( + &clip->tracking, width, height, undist_marker.pos, undist_marker.pos); - IMB_freeImBuf(ibuf); + undist_marker.pos[0] /= width; + undist_marker.pos[1] /= height * aspy; } - if ((track->flag & TRACK_LOCKED) == 0) { - float pat_min[2], pat_max[2]; + scopes->track_search = BKE_tracking_get_search_imbuf( + ibuf, track, &undist_marker, true, true); - scopes->track_locked = false; + scopes->undist_marker = undist_marker; - /* XXX: would work fine with non-transformed patterns, but would likely fail - * with transformed patterns, but that would be easier to debug when - * we'll have real pattern sampling (at least to test) */ - BKE_tracking_marker_pattern_minmax(marker, pat_min, pat_max); + scopes->frame_width = ibuf->x; + scopes->frame_height = ibuf->y; - scopes->slide_scale[0] = pat_max[0] - pat_min[0]; - scopes->slide_scale[1] = pat_max[1] - pat_min[1]; - } + scopes->use_track_mask = (track->flag & TRACK_PREVIEW_ALPHA) != 0; } + + IMB_freeImBuf(ibuf); } - scopes->framenr = user->framenr; - scopes->ok = true; + if ((track->flag & TRACK_LOCKED) == 0) { + float pat_min[2], pat_max[2]; + + scopes->track_locked = false; + + /* XXX: would work fine with non-transformed patterns, but would likely fail + * with transformed patterns, but that would be easier to debug when + * we'll have real pattern sampling (at least to test) */ + BKE_tracking_marker_pattern_minmax(marker, pat_min, pat_max); + + scopes->slide_scale[0] = pat_max[0] - pat_min[0]; + scopes->slide_scale[1] = pat_max[1] - pat_min[1]; + } } static void movieclip_build_proxy_ibuf( |