diff options
| author | Sybren A. Stüvel <sybren@blender.org> | 2022-01-06 13:37:31 +0300 |
|---|---|---|
| committer | Sybren A. Stüvel <sybren@blender.org> | 2022-01-06 13:41:03 +0300 |
| commit | f9aa6376f1a2b13ab12a77197257e9145f029792 (patch) | |
| tree | 8ef47adab543467512f0795f869793078c5dbdd8 /source/blender/blenkernel/BKE_nla.h | |
| parent | 1484fe260be9601fe88a21225e05483dc0c280d9 (diff) | |
Cleanup: anim, remove `const` declarations from pass-by-value params
Remove `const` from pass-by-value parameters in function declarations.
The variables passed as parameters can never be modified by the function
anyway, so declaring them as `const` is meaningless. Having the
declaration there could confuse, especially as it suggests it does have
a meaning, training people to write meaningless code.
Diffstat (limited to 'source/blender/blenkernel/BKE_nla.h')
| -rw-r--r-- | source/blender/blenkernel/BKE_nla.h | 20 |
1 files changed, 9 insertions, 11 deletions
diff --git a/source/blender/blenkernel/BKE_nla.h b/source/blender/blenkernel/BKE_nla.h index b297851ad90..61431547bfb 100644 --- a/source/blender/blenkernel/BKE_nla.h +++ b/source/blender/blenkernel/BKE_nla.h @@ -71,8 +71,8 @@ void BKE_nla_tracks_free(ListBase *tracks, bool do_id_user); */ struct NlaStrip *BKE_nlastrip_copy(struct Main *bmain, struct NlaStrip *strip, - const bool use_same_action, - const int flag); + bool use_same_action, + int flag); /** * Copy a single NLA Track. * \param flag: Control ID pointers management, see LIB_ID_CREATE_.../LIB_ID_COPY_... @@ -80,14 +80,14 @@ struct NlaStrip *BKE_nlastrip_copy(struct Main *bmain, */ struct NlaTrack *BKE_nlatrack_copy(struct Main *bmain, struct NlaTrack *nlt, - const bool use_same_actions, - const int flag); + bool use_same_actions, + int flag); /** * Copy all NLA data. * \param flag: Control ID pointers management, see LIB_ID_CREATE_.../LIB_ID_COPY_... * flags in BKE_lib_id.h */ -void BKE_nla_tracks_copy(struct Main *bmain, ListBase *dst, const ListBase *src, const int flag); +void BKE_nla_tracks_copy(struct Main *bmain, ListBase *dst, const ListBase *src, int flag); /** * Copy NLA tracks from #adt_source to #adt_dest, and update the active track/strip pointers to @@ -115,7 +115,7 @@ struct NlaStrip *BKE_nlastrip_new(struct bAction *act); */ struct NlaStrip *BKE_nlastack_add_strip(struct AnimData *adt, struct bAction *act, - const bool is_liboverride); + bool is_liboverride); /** * Add a NLA Strip referencing the given speaker's sound. */ @@ -217,9 +217,7 @@ void BKE_nlatrack_sort_strips(struct NlaTrack *nlt); * Add the given NLA-Strip to the given NLA-Track, assuming that it * isn't currently attached to another one. */ -bool BKE_nlatrack_add_strip(struct NlaTrack *nlt, - struct NlaStrip *strip, - const bool is_liboverride); +bool BKE_nlatrack_add_strip(struct NlaTrack *nlt, struct NlaStrip *strip, bool is_liboverride); /** * Get the extents of the given NLA-Track including gaps between strips, @@ -309,7 +307,7 @@ bool BKE_nla_action_is_stashed(struct AnimData *adt, struct bAction *act); * "Stash" an action (i.e. store it as a track/layer in the NLA, but non-contributing) * to retain it in the file for future uses. */ -bool BKE_nla_action_stash(struct AnimData *adt, const bool is_liboverride); +bool BKE_nla_action_stash(struct AnimData *adt, bool is_liboverride); /* ............ */ @@ -321,7 +319,7 @@ bool BKE_nla_action_stash(struct AnimData *adt, const bool is_liboverride); * * TODO: maybe we should have checks for this too. */ -void BKE_nla_action_pushdown(struct AnimData *adt, const bool is_liboverride); +void BKE_nla_action_pushdown(struct AnimData *adt, bool is_liboverride); /** * Find the active strip + track combination, and set them up as the tweaking track, |