Multi Object Mode: ED_operator_editmesh
AbandonedPublic
Actions

Authored by Habib Gahbiche (zazizizou) on Jun 2 2018, 2:10 PM.

Details

Reviewers: None
Maniphest Tasks: T54643: Multi-Object-Mode: EditMesh Tools

Summary

Currently, an operator checks if it can be applied to only one object. See ot->poll = ED_operator_editmesh. With this patch, an operator will verify if the it can be applied to all selected objects.

This change is small but it will affect hundreds of operators so feedback on this (e.g. whether the change is necessary in the first place, etc..) is very much appreciated.

Diff Detail

Repository: rB Blender

Event Timeline

Habib Gahbiche (zazizizou) created this revision.Jun 2 2018, 2:10 PM

Habib Gahbiche (zazizizou) added a task: T54643: Multi-Object-Mode: EditMesh Tools.Jun 2 2018, 2:13 PM

BKE_editmesh_from_object is called only for objects of type OB_MESH

Habib Gahbiche (zazizizou) mentioned this in T54643: Multi-Object-Mode: EditMesh Tools.Jun 2 2018, 3:03 PM

Poll functions run very often (on every buttons redraw), having them allocate & search is going to slow down Blender too much.

We will keep the convention of using the active object to define which mode/type is currently edited.

In D3456#80196, @Campbell Barton (campbellbarton) wrote:

Poll functions run very often (on every buttons redraw), having them allocate & search is going to slow down Blender too much.

We will keep the convention of using the active object to define which mode/type is currently edited.

Ok, thanks for the feedback!

Revision Contents
Changeset List

Path

Size

source/

blender/

editors/

screen/

screen_ops.c

18 lines

Diff 11181

View Options

source/blender/editors/screen/screen_ops.c

Multi Object Mode: ED_operator_editmesh AbandonedPublicActions