Paths

Table of Contentst

Differential D3382

T54643-Multi-Object-Mode: Edit Curve Tools: CURVE_OT_subdivide
AbandonedPublic
Actions

Authored by Daniel Griffin (dgriffin91) on May 16 2018, 8:28 AM.

Tags

None

Subscribers

Campbell Barton (campbellbarton)

Details

Reviewers

Dalai Felinto (dfelinto)
Campbell Barton (campbellbarton)

Maniphest Tasks

T54648: Multi-Object-Mode: Edit Curve Tools

Summary

T54643-Multi-Object-Mode: Edit Curve Tools: CURVE_OT_subdivide

Diff Detail

Event Timeline

Daniel Griffin (dgriffin91) created this revision.May 16 2018, 8:28 AM

Daniel Griffin (dgriffin91) updated this revision to Diff 10964.May 16 2018, 8:33 AM

Dalai Felinto (dfelinto) requested changes to this revision.May 16 2018, 9:54 AM

Dalai Felinto (dfelinto) added inline comments.

source/blender/editors/curve/editcurve.c
3380	Skip the for loop when no selected element.
3381	Keep all the `RNA_.*_get` out of the for loop.

This revision now requires changes to proceed.May 16 2018, 9:54 AM

Moved RNA_int_get out of loop.

Skipping for loop when no selected element.

Daniel Griffin (dgriffin91) added tasks: T54648: Multi-Object-Mode: Edit Curve Tools, T54641: Multi-object mode support (parent task).May 23 2018, 7:10 AM

Campbell Barton (campbellbarton) mentioned this in rB17aef80207bc: Multi-Object-Mode: EditCurve Subdivide.May 23 2018, 7:42 AM

Thanks for the patch, committed rB17aef80207bcff8bfb58e4da890b7f1310318a4c

Campbell Barton (campbellbarton) abandoned this revision.May 23 2018, 7:45 AM

Campbell Barton (campbellbarton) mentioned this in D3288: Multi-Object-Mode: Edit Curve Tools add support for Subdivide.May 23 2018, 7:56 AM

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

Revision Contents
Changeset List

Path

Size

source/

blender/

editors/

curve/

22 lines

Diff 10964

source/blender/editors/curve/editcurve.c

Loading...