Fix T58928: New Action causes crash when duplicating bone
authorSergey Sharybin <sergey.vfx@gmail.com>
Mon, 10 Dec 2018 14:59:15 +0000 (15:59 +0100)
committerSergey Sharybin <sergey.vfx@gmail.com>
Mon, 10 Dec 2018 14:59:15 +0000 (15:59 +0100)
commit9fa408f51b70169d4e1ecf0801a3b9f726a9530b
tree07bdf9f3a97df3dd2083abd358df3d66555b983b
parentd7e7c4ddf704589d16f38f87c4055055e6703a9c
Fix T58928: New Action causes crash when duplicating bone

Basically, armature update is not supposed to be run in edit mode.
This worked in master and new dependency graph because nobody was
tagging armature for an update.

But with all those copy-on-write and other things we can't ensure
tag doesn't happen (and we shouldn't). So now we ensure unwanted
code is not run from the code itself.

P.S. Deeper reason of this goes to the optimization of not updating
pose channels when in edit mode. Since pose doesn't define anything
there we don't want to be bothered with a pose update after every
operation which changes it.
source/blender/blenkernel/intern/armature_update.c