Page MenuHome
Differential D3287

Multi-Object-Mode: Edit Curve Tools add support for Switch Direction
AbandonedPublic
Actions

Authored by Guy Boicey (stealthtech) on May 9 2018, 7:13 AM.

Details

Reviewers
Dalai Felinto (dfelinto)
Maniphest Tasks
T54648: Multi-Object-Mode: Edit Curve Tools
Summary

Multi-Object-Mode: Edit Curve Tools add support for Switch Direction (CURVE_OT_switch_direction)

Diff Detail

Event Timeline

Guy Boicey (stealthtech) created this revision.May 9 2018, 7:13 AM
Guy Boicey (stealthtech) added a task: T54648: Multi-Object-Mode: Edit Curve Tools.May 9 2018, 7:25 AM
Dalai Felinto (dfelinto) requested changes to this revision.May 25 2018, 3:01 PM
Dalai Felinto (dfelinto) added inline comments.
source/blender/editors/curve/editcurve.c
2315

Mind moving this int declaration closer to where you use it? It seems a bit out of place now.

2321

Move { to the same line as the for, after the ( with a space between them.

2337

If there is no selected curve you still call DEG update and modifier, and ...
Look at the changed test here: rB647232f7f16c.

This revision now requires changes to proceed.May 25 2018, 3:01 PM
Dalai Felinto (dfelinto) abandoned this revision.Nov 2 2018, 6:20 PM

Thanks, the same functionality is already committed, closing the patch now.

Revision Contents
Changeset List

PathSize
source/
blender/
editors/
curve/
39 lines

Diff 10680

View Options

source/blender/editors/curve/editcurve.c

Loading...