Differential D9693

Nla Refactor: nlaevalchan_validate_index_ex()
ClosedPublic
Actions

Authored by Wayde Moss (GuiltyGhost) on Dec 1 2020, 12:16 PM.

Tags

Subscribers

None

Details

Reviewers

Sybren A. Stüvel (sybren)

Commits

rBfa6bf8f5b65e: Nla Refactor: nlaevalchan_validate_index_ex()

Summary

And removes redundant index local variable. Future patches D8296: Feature: NLA: Evaluate Whole NLA Stack in Tweak Mode and D8867: Feature: NLA Merge Strips make use of this function.

No functional changes.
Split from D9247: Refactor: NLA Prep for D8296

Diff Detail

Repository

Branch

refactor2_nlaevalchan_validate_index_ex (branched from master)

Build Status

Buildable 11481
Build 11481: arc lint + arc unit

Event Timeline

Wayde Moss (GuiltyGhost) created this revision.Dec 1 2020, 12:16 PM

Harbormaster completed remote builds in B11481: Diff 31486.Dec 1 2020, 12:16 PM

Wayde Moss (GuiltyGhost) requested review of this revision.Dec 1 2020, 12:16 PM

Wayde Moss (GuiltyGhost) added a project: Animation & Rigging.Dec 1 2020, 12:17 PM

Sybren A. Stüvel (sybren) mentioned this in rBL62523: Constraints: add test case for D9693: Custom Space.Dec 3 2020, 11:30 AM

LGTM.

This revision is now accepted and ready to land.Dec 3 2020, 12:04 PM

Closed by commit rBfa6bf8f5b65e: Nla Refactor: nlaevalchan_validate_index_ex() (authored by Wayde Moss (GuiltyGhost)). · Explain WhyDec 14 2020, 11:40 PM

This revision was automatically updated to reflect the committed changes.

Wayde Moss (GuiltyGhost) added a commit: rBfa6bf8f5b65e: Nla Refactor: nlaevalchan_validate_index_ex().

Revision Contents
Changeset List

Path

Size

source/

blender/

blenkernel/

intern/

44 lines

Diff 31486

source/blender/blenkernel/intern/anim_sys.c

Loading...