Page MenuHome
Differential D3872

Edit Curve Tools CURVE_OT_hide
AbandonedPublic
Actions

Authored by Habib Gahbiche (zazizizou) on Oct 31 2018, 10:54 PM.

Details

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

Diff Detail

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

Event Timeline

Habib Gahbiche (zazizizou) created this revision.Oct 31 2018, 10:54 PM
Habib Gahbiche (zazizizou) added a task: T54648: Multi-Object-Mode: Edit Curve Tools.
Habib Gahbiche (zazizizou) added reviewers: Campbell Barton (campbellbarton), Dalai Felinto (dfelinto).Oct 31 2018, 10:56 PM
Habib Gahbiche (zazizizou) added inline comments.Oct 31 2018, 10:58 PM
source/blender/editors/curve/editcurve.c
2899

I don't think it's necessary to skip the loop if there are no selected control points, since we iterate over all the points to check for selection anyway.

Habib Gahbiche (zazizizou) updated this revision to Diff 12339.Nov 1 2018, 10:33 AM

free objects

Dalai Felinto (dfelinto) abandoned this revision.Nov 2 2018, 6:17 PM

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

Revision Contents
Changeset List

PathSize
source/
blender/
editors/
curve/
106 lines

Diff 12339

View Options

source/blender/editors/curve/editcurve.c

Loading...