From 976526559d2690dfeda19acc6b82c6767f21d2eb Mon Sep 17 00:00:00 2001 From: Bastien Montagne Date: Tue, 29 Jul 2014 15:02:28 +0200 Subject: Fix T41228: Selection of bones bug. Turns out to be mostly some cleanup in Pose select code, got rid of magic numbers (now understand usual SEL_xxx enums) in ED_pose_deselectall(), which was renamed to ED_pose_de_selectall, and have a new bool parameter to ignore visibility status in its process (was the root of the reported issue). Also factorized slightly "(de)select all" code. Yet this area could use much more cleanup probably... --- source/blender/editors/animation/anim_channels_edit.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'source/blender/editors/animation') diff --git a/source/blender/editors/animation/anim_channels_edit.c b/source/blender/editors/animation/anim_channels_edit.c index f578a57c707..97078043dc9 100644 --- a/source/blender/editors/animation/anim_channels_edit.c +++ b/source/blender/editors/animation/anim_channels_edit.c @@ -2802,7 +2802,7 @@ static int mouse_anim_channels(bContext *C, bAnimContext *ac, int channel_index, /* deselect all other channels */ ANIM_deselect_anim_channels(ac, ac->data, ac->datatype, false, ACHANNEL_SETFLAG_CLEAR); - if (pchan) ED_pose_deselectall(ob, 0); + if (pchan) ED_pose_de_selectall(ob, SEL_DESELECT, false); /* only select channels in group and group itself */ for (fcu = agrp->channels.first; fcu && fcu->grp == agrp; fcu = fcu->next) @@ -2812,7 +2812,7 @@ static int mouse_anim_channels(bContext *C, bAnimContext *ac, int channel_index, else { /* select group by itself */ ANIM_deselect_anim_channels(ac, ac->data, ac->datatype, false, ACHANNEL_SETFLAG_CLEAR); - if (pchan) ED_pose_deselectall(ob, 0); + if (pchan) ED_pose_de_selectall(ob, SEL_DESELECT, false); agrp->flag |= AGRP_SELECTED; } -- cgit v1.2.3