Commit: 1e0654f162afdc75c4e4286e2bd8d276fe668f78
Author: Brecht Van Lommel
Date:   Wed Jan 29 16:11:34 2014 +0100
https://developer.blender.org/rB1e0654f162afdc75c4e4286e2bd8d276fe668f78

Fix T38393: bone roll not restored properly after cancelling rotation of bones.

===================================================================

M       source/blender/editors/transform/transform_conversions.c
M       source/blender/editors/transform/transform_generics.c

===================================================================

diff --git a/source/blender/editors/transform/transform_conversions.c 
b/source/blender/editors/transform/transform_conversions.c
index 47bd527..9be83bf 100644
--- a/source/blender/editors/transform/transform_conversions.c
+++ b/source/blender/editors/transform/transform_conversions.c
@@ -1197,6 +1197,7 @@ static void createTransArmatureVerts(TransInfo *t)
 
                                        if ((ebo->flag & BONE_ROOTSEL) == 0) {
                                                td->extra = ebo;
+                                               td->ival = ebo->roll;
                                        }
 
                                        td->ext = NULL;
@@ -1219,6 +1220,7 @@ static void createTransArmatureVerts(TransInfo *t)
                                        ED_armature_ebone_to_mat3(ebo, 
td->axismtx);
 
                                        td->extra = ebo; /* to fix roll */
+                                       td->ival = ebo->roll;
 
                                        td->ext = NULL;
                                        td->val = NULL;
diff --git a/source/blender/editors/transform/transform_generics.c 
b/source/blender/editors/transform/transform_generics.c
index 55ecc53..8bf3e65 100644
--- a/source/blender/editors/transform/transform_generics.c
+++ b/source/blender/editors/transform/transform_generics.c
@@ -806,21 +806,28 @@ static void recalcData_view3d(TransInfo *t)
                                                float roll;
                                                
                                                ebo = td->extra;
-                                               copy_v3_v3(up_axis, 
td->axismtx[2]);
-                                               
-                                               if (t->mode != TFM_ROTATION) {
-                                                       sub_v3_v3v3(vec, 
ebo->tail, ebo->head);
-                                                       normalize_v3(vec);
-                                                       
rotation_between_vecs_to_quat(qrot, td->axismtx[1], vec);
-                                                       mul_qt_v3(qrot, 
up_axis);
+
+                                               if (t->state == TRANS_CANCEL) {
+                                                       /* restore roll */
+                                                       ebo->roll = td->ival;
                                                }
                                                else {
-                                                       mul_m3_v3(t->mat, 
up_axis);
+                                                       copy_v3_v3(up_axis, 
td->axismtx[2]);
+                                                       
+                                                       if (t->mode != 
TFM_ROTATION) {
+                                                               
sub_v3_v3v3(vec, ebo->tail, ebo->head);
+                                                               
normalize_v3(vec);
+                                                               
rotation_between_vecs_to_quat(qrot, td->axismtx[1], vec);
+                                                               mul_qt_v3(qrot, 
up_axis);
+                                                       }
+                                                       else {
+                                                               
mul_m3_v3(t->mat, up_axis);
+                                                       }
+                                                       
+                                                       /* roll has a tendency 
to flip in certain orientations - [#34283], [#33974] */
+                                                       roll = 
ED_rollBoneToVector(ebo, up_axis, false);
+                                                       ebo->roll = 
angle_compat_rad(roll, ebo->roll);
                                                }
-                                               
-                                               /* roll has a tendency to flip 
in certain orientations - [#34283], [#33974] */
-                                               roll = ED_rollBoneToVector(ebo, 
up_axis, false);
-                                               ebo->roll = 
angle_compat_rad(roll, ebo->roll);
                                        }
                                }
                        }

_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to