Page MenuHome
Differential D3311

ARMATURE_OT_switch_direction , multi-object Task T54646 , https://developer.blender.org/T54646
AbandonedPublic
Actions

Authored by Harsha (blendcode) on May 10 2018, 2:01 AM.

Details

Reviewers
Dalai Felinto (dfelinto)
Maniphest Tasks
T54646: Multi-Object-Mode: Edit Armature Tools

Diff Detail

Repository
rB Blender
Branch
testbranch (branched from blender2.8)
Build Status
Buildable 1524
Build 1524: arc lint + arc unit

Event Timeline

Harsha (blendcode) created this revision.May 10 2018, 2:01 AM
Harsha (blendcode) added a task: T54646: Multi-Object-Mode: Edit Armature Tools.May 10 2018, 2:02 AM
Harsha (blendcode) added a task: T54641: Multi-object mode support (parent task).
Dalai Felinto (dfelinto) abandoned this revision.Sep 29 2018, 12:13 AM

Thanks for the patch.
Committed something similar on rB0f9b7560db36eaf1ce4d7e3e0c116f2f39e3227c.

I didn't incorporate your check for selected bones though. People should select things before running the operators, no?

source/blender/editors/armature/armature_edit.c
1061

You can't return in the middle of the loop. Not only this will lead to mem leakage but also other selected objects may still need to be processed.

Dalai Felinto (dfelinto) removed a task: T54641: Multi-object mode support (parent task).Oct 30 2018, 7:44 PM

Revision Contents
Changeset List

PathSize
source/
blender/
editors/
armature/
168 lines

Diff 10762

View Options

source/blender/editors/armature/armature_edit.c

Loading...