diff options
| author | Joshua Leung <aligorith@gmail.com> | 2009-09-20 15:21:44 +0400 |
|---|---|---|
| committer | Joshua Leung <aligorith@gmail.com> | 2009-09-20 15:21:44 +0400 |
| commit | ad25fc829e736be4bbf2e2a125347eda294fdac9 (patch) | |
| tree | 40f148482c2248c613f1302dcf00201a4795ce94 /source/blender/editors/armature | |
| parent | 2c871f722d3cbc80c9f076d7c23137e19633ba3a (diff) | |
2.5 - More animation/rigging bugfixes
* #19419: PoseLib rename/remove tools could crash when an invalid
(However, now care is needed when touching that index field, since the warnings can keep piling up)
* Added Browse Poses for PoseLib to the toolbar
* Removing constraints from bones now properly updates. A DAG rebuild is now forced, and the constraint flags are cleared.
* Attempting to improve the situation with Copy Rotation constraint and rotation orders other than xyz. Unforunately, it looks like a different method is required...
Diffstat (limited to 'source/blender/editors/armature')
| -rw-r--r-- | source/blender/editors/armature/poselib.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source/blender/editors/armature/poselib.c b/source/blender/editors/armature/poselib.c index ce07cfb9042..386cb6512a3 100644 --- a/source/blender/editors/armature/poselib.c +++ b/source/blender/editors/armature/poselib.c @@ -467,6 +467,7 @@ static int poselib_remove_exec (bContext *C, wmOperator *op) marker= BLI_findlink(&act->markers, RNA_int_get(op->ptr, "index")); if (marker == NULL) { BKE_report(op->reports, RPT_ERROR, "Invalid index for Pose"); + return OPERATOR_CANCELLED; } /* remove relevant keyframes */ @@ -533,6 +534,7 @@ static int poselib_rename_exec (bContext *C, wmOperator *op) marker= BLI_findlink(&act->markers, RNA_int_get(op->ptr, "index")); if (marker == NULL) { BKE_report(op->reports, RPT_ERROR, "Invalid index for Pose"); + return OPERATOR_CANCELLED; } /* get new name */ |