[Bf-blender-cvs] [adb371b9e12] tracking_refactor_v3: Refactor: Remove trivial accessor functions from tracking
Sergey Sharybin
noreply at git.blender.org
Tue Oct 18 12:18:01 CEST 2022
Commit: adb371b9e1290304b916586b6edae3b493546468
Author: Sergey Sharybin
Date: Wed Oct 5 15:56:08 2022 +0200
Branches: tracking_refactor_v3
https://developer.blender.org/rBadb371b9e1290304b916586b6edae3b493546468
Refactor: Remove trivial accessor functions from tracking
===================================================================
M source/blender/blenkernel/BKE_tracking.h
M source/blender/blenkernel/intern/tracking.c
M source/blender/blenkernel/intern/tracking_solver.c
M source/blender/blenloader/intern/versioning_260.c
M source/blender/blenloader/intern/versioning_270.c
M source/blender/compositor/operations/COM_KeyingScreenOperation.cc
M source/blender/draw/engines/overlay/overlay_extra.cc
M source/blender/editors/space_clip/clip_dopesheet_ops.c
M source/blender/editors/space_clip/tracking_ops.c
M source/blender/editors/space_clip/tracking_ops_orient.cc
M source/blender/editors/space_view3d/view3d_select.cc
M source/blender/editors/space_view3d/view3d_snap.c
M source/blender/editors/transform/transform_snap_object.cc
M source/blender/makesrna/intern/rna_tracking.c
===================================================================
diff --git a/source/blender/blenkernel/BKE_tracking.h b/source/blender/blenkernel/BKE_tracking.h
index 4f4cc86ed6d..243c4980651 100644
--- a/source/blender/blenkernel/BKE_tracking.h
+++ b/source/blender/blenkernel/BKE_tracking.h
@@ -451,13 +451,6 @@ struct MovieTrackingObject *BKE_tracking_object_get_named(struct MovieTracking *
struct MovieTrackingObject *BKE_tracking_object_get_active(const struct MovieTracking *tracking);
struct MovieTrackingObject *BKE_tracking_object_get_camera(const struct MovieTracking *tracking);
-struct ListBase *BKE_tracking_object_get_tracks(struct MovieTracking *tracking,
- struct MovieTrackingObject *object);
-struct ListBase *BKE_tracking_object_get_plane_tracks(struct MovieTracking *tracking,
- struct MovieTrackingObject *object);
-struct MovieTrackingReconstruction *BKE_tracking_object_get_reconstruction(
- struct MovieTracking *tracking, struct MovieTrackingObject *object);
-
/* --------------------------------------------------------------------
* Camera.
*/
diff --git a/source/blender/blenkernel/intern/tracking.c b/source/blender/blenkernel/intern/tracking.c
index b953c160dbc..5036218b1a9 100644
--- a/source/blender/blenkernel/intern/tracking.c
+++ b/source/blender/blenkernel/intern/tracking.c
@@ -372,8 +372,7 @@ ListBase *BKE_tracking_get_active_plane_tracks(MovieTracking *tracking)
MovieTrackingReconstruction *BKE_tracking_get_active_reconstruction(MovieTracking *tracking)
{
MovieTrackingObject *object = BKE_tracking_object_get_active(tracking);
-
- return BKE_tracking_object_get_reconstruction(tracking, object);
+ return &object->reconstruction;
}
void BKE_tracking_get_camera_object_matrix(Object *camera_object, float mat[4][4])
@@ -465,23 +464,19 @@ void BKE_tracking_clipboard_free(void)
BLI_listbase_clear(&tracking_clipboard.tracks);
}
-void BKE_tracking_clipboard_copy_tracks(MovieTracking *tracking, MovieTrackingObject *object)
+void BKE_tracking_clipboard_copy_tracks(MovieTracking *UNUSED(tracking),
+ MovieTrackingObject *object)
{
- ListBase *tracksbase = BKE_tracking_object_get_tracks(tracking, object);
- MovieTrackingTrack *track = tracksbase->first;
-
/* First drop all tracks from current clipboard. */
BKE_tracking_clipboard_free();
/* Then copy all selected visible tracks to it. */
- while (track) {
+ LISTBASE_FOREACH (MovieTrackingTrack *, track, &object->tracks) {
if (TRACK_SELECTED(track) && (track->flag & TRACK_HIDDEN) == 0) {
MovieTrackingTrack *new_track = BKE_tracking_track_duplicate(track);
BLI_addtail(&tracking_clipboard.tracks, new_track);
}
-
- track = track->next;
}
}
@@ -490,12 +485,10 @@ bool BKE_tracking_clipboard_has_tracks(void)
return (BLI_listbase_is_empty(&tracking_clipboard.tracks) == false);
}
-void BKE_tracking_clipboard_paste_tracks(MovieTracking *tracking, MovieTrackingObject *object)
+void BKE_tracking_clipboard_paste_tracks(MovieTracking *UNUSED(tracking),
+ MovieTrackingObject *object)
{
- ListBase *tracksbase = BKE_tracking_object_get_tracks(tracking, object);
- MovieTrackingTrack *track = tracking_clipboard.tracks.first;
-
- while (track) {
+ LISTBASE_FOREACH (MovieTrackingTrack *, track, &tracking_clipboard.tracks) {
MovieTrackingTrack *new_track = BKE_tracking_track_duplicate(track);
/* TODO(sergey): Preserve active track from before the copy. */
@@ -503,10 +496,8 @@ void BKE_tracking_clipboard_paste_tracks(MovieTracking *tracking, MovieTrackingO
object->active_track = new_track;
}
- BLI_addtail(tracksbase, new_track);
- BKE_tracking_track_unique_name(tracksbase, new_track);
-
- track = track->next;
+ BLI_addtail(&object->tracks, new_track);
+ BKE_tracking_track_unique_name(&object->tracks, new_track);
}
}
@@ -1041,19 +1032,14 @@ void BKE_tracking_tracks_average(MovieTrackingTrack *dst_track,
tracking_average_tracks(dst_track, src_tracks, num_src_tracks);
}
-MovieTrackingTrack *BKE_tracking_track_get_named(MovieTracking *tracking,
+MovieTrackingTrack *BKE_tracking_track_get_named(MovieTracking *UNUSED(tracking),
MovieTrackingObject *object,
const char *name)
{
- ListBase *tracksbase = BKE_tracking_object_get_tracks(tracking, object);
- MovieTrackingTrack *track = tracksbase->first;
-
- while (track) {
+ LISTBASE_FOREACH (MovieTrackingTrack *, track, &object->tracks) {
if (STREQ(track->name, name)) {
return track;
}
-
- track = track->next;
}
return NULL;
@@ -1063,28 +1049,19 @@ MovieTrackingTrack *BKE_tracking_track_get_indexed(MovieTracking *tracking,
int tracknr,
ListBase **r_tracksbase)
{
- MovieTrackingObject *object;
int cur = 1;
- object = tracking->objects.first;
- while (object) {
- ListBase *tracksbase = BKE_tracking_object_get_tracks(tracking, object);
- MovieTrackingTrack *track = tracksbase->first;
-
- while (track) {
+ LISTBASE_FOREACH (MovieTrackingObject *, object, &tracking->objects) {
+ LISTBASE_FOREACH (MovieTrackingTrack *, track, &object->tracks) {
if (track->flag & TRACK_HAS_BUNDLE) {
if (cur == tracknr) {
- *r_tracksbase = tracksbase;
+ *r_tracksbase = &object->tracks;
return track;
}
cur++;
}
-
- track = track->next;
}
-
- object = object->next;
}
*r_tracksbase = NULL;
@@ -1701,13 +1678,11 @@ void BKE_tracking_plane_track_free(MovieTrackingPlaneTrack *plane_track)
MEM_freeN(plane_track->point_tracks);
}
-MovieTrackingPlaneTrack *BKE_tracking_plane_track_get_named(MovieTracking *tracking,
+MovieTrackingPlaneTrack *BKE_tracking_plane_track_get_named(MovieTracking *UNUSED(tracking),
MovieTrackingObject *object,
const char *name)
{
- ListBase *plane_tracks_base = BKE_tracking_object_get_plane_tracks(tracking, object);
-
- LISTBASE_FOREACH (MovieTrackingPlaneTrack *, plane_track, plane_tracks_base) {
+ LISTBASE_FOREACH (MovieTrackingPlaneTrack *, plane_track, &object->plane_tracks) {
if (STREQ(plane_track->name, name)) {
return plane_track;
}
@@ -2098,24 +2073,6 @@ MovieTrackingObject *BKE_tracking_object_get_camera(const MovieTracking *trackin
return NULL;
}
-ListBase *BKE_tracking_object_get_tracks(MovieTracking *UNUSED(tracking),
- MovieTrackingObject *object)
-{
- return &object->tracks;
-}
-
-ListBase *BKE_tracking_object_get_plane_tracks(MovieTracking *UNUSED(tracking),
- MovieTrackingObject *object)
-{
- return &object->plane_tracks;
-}
-
-MovieTrackingReconstruction *BKE_tracking_object_get_reconstruction(
- MovieTracking *UNUSED(tracking), MovieTrackingObject *object)
-{
- return &object->reconstruction;
-}
-
/* --------------------------------------------------------------------
* Camera.
*/
@@ -2227,15 +2184,12 @@ void BKE_tracking_camera_to_blender(
BKE_tracking_camera_shift_get(tracking, width, height, &camera->shiftx, &camera->shifty);
}
-MovieReconstructedCamera *BKE_tracking_camera_get_reconstructed(MovieTracking *tracking,
+MovieReconstructedCamera *BKE_tracking_camera_get_reconstructed(MovieTracking *UNUSED(tracking),
MovieTrackingObject *object,
int framenr)
{
- MovieTrackingReconstruction *reconstruction;
- int a;
-
- reconstruction = BKE_tracking_object_get_reconstruction(tracking, object);
- a = reconstructed_camera_index_get(reconstruction, framenr, false);
+ MovieTrackingReconstruction *reconstruction = &object->reconstruction;
+ int a = reconstructed_camera_index_get(reconstruction, framenr, false);
if (a == -1) {
return NULL;
@@ -2244,18 +2198,15 @@ MovieReconstructedCamera *BKE_tracking_camera_get_reconstructed(MovieTracking *t
return &reconstruction->cameras[a];
}
-void BKE_tracking_camera_get_reconstructed_interpolate(MovieTracking *tracking,
+void BKE_tracking_camera_get_reconstructed_interpolate(MovieTracking *UNUSED(tracking),
MovieTrackingObject *object,
float framenr,
float mat[4][4])
{
- MovieTrackingReconstruction *reconstruction;
- MovieReconstructedCamera *cameras;
- int a;
+ MovieTrackingReconstruction *reconstruction = &object->reconstruction;
+ MovieReconstructedCamera *cameras = reconstruction->cameras;
- reconstruction = BKE_tracking_object_get_reconstruction(tracking, object);
- cameras = reconstruction->cameras;
- a = reconstructed_camera_index_get(reconstruction, (int)framenr, true);
+ int a = reconstructed_camera_index_get(reconstruction, (int)framenr, true);
if (a == -1) {
unit_m4(mat);
@@ -3208,14 +3159,12 @@ static void tracking_dopesheet_channels_calc(MovieTracking *tracking)
{
MovieTrackingObject *object = BKE_tracking_object_get_active(tracking);
MovieTrackingDopesheet *dopesheet = &tracking->dopesheet;
- MovieTrackingReconstruction *reconstruction = BKE_tracking_object_get_reconstruction(tracking,
- object);
- ListBase *tracksbase = BKE_tracking_object_get_tracks(tracking, object);
+ MovieTrackingReconstruction *reconstruction = &object->reconstruction;
bool sel_only = (dopesheet->flag & TRACKING_DOPE_SELECTED_ONLY) != 0;
bool show_hidden = (dopesheet->flag & TRACKING_DOPE_SHOW_HIDDEN) != 0;
- LISTBASE_FOREACH (MovieTrackingTrack *, track, tracksbase) {
+ LISTBASE_FOREACH (MovieTrackingTrack *, track, &object->tracks) {
if (!show_hid
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list