diff options
author | Brecht Van Lommel <brechtvanlommel@gmail.com> | 2013-11-26 03:38:50 +0400 |
---|---|---|
committer | Brecht Van Lommel <brechtvanlommel@gmail.com> | 2013-11-29 20:49:01 +0400 |
commit | cedc90b6077a01a60f589f09a5c82f4b7a359a4b (patch) | |
tree | e6c24bf3fdb0aa2286c744e30ce45b10c8cb0085 /source/blender/editors/mask | |
parent | 77719bfd0669cc675ad729f4c51672173842faca (diff) |
Fix errors and inconsistencies in confirmation popup removal.
* Improve some clip editor messages
* Remove popup for metastrips, seems unnecessary
* Renamed some variables for consistency
* Avoid unnecessary call to CTX_DATA_COUNT
Reviewed By: sergey, campbellbarton, aligorith
Differential Revision: http://developer.blender.org/D44
Diffstat (limited to 'source/blender/editors/mask')
-rw-r--r-- | source/blender/editors/mask/mask_ops.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/source/blender/editors/mask/mask_ops.c b/source/blender/editors/mask/mask_ops.c index a4943373288..7d7960d2e86 100644 --- a/source/blender/editors/mask/mask_ops.c +++ b/source/blender/editors/mask/mask_ops.c @@ -1036,19 +1036,17 @@ static int delete_exec(bContext *C, wmOperator *op) } } - if (num_deleted == 0) { + if (num_deleted == 0) return OPERATOR_CANCELLED; - } - else { - /* TODO: only update edited splines */ - BKE_mask_update_display(mask, CFRA); - WM_event_add_notifier(C, NC_MASK | NA_EDITED, mask); + /* TODO: only update edited splines */ + BKE_mask_update_display(mask, CFRA); - BKE_reportf(op->reports, RPT_INFO, "Deleted selected control points from mask '%s'", mask->id.name); + WM_event_add_notifier(C, NC_MASK | NA_EDITED, mask); - return OPERATOR_FINISHED; - } + BKE_reportf(op->reports, RPT_INFO, "Deleted %d control points from mask '%s'", num_deleted, mask->id.name); + + return OPERATOR_FINISHED; } void MASK_OT_delete(wmOperatorType *ot) |