diff options
| author | Bastien Montagne <montagne29@wanadoo.fr> | 2012-06-19 20:26:01 +0400 |
|---|---|---|
| committer | Bastien Montagne <montagne29@wanadoo.fr> | 2012-06-19 20:26:01 +0400 |
| commit | 8796c3eff1c3019513fbf3a00e573a24fcd56a7d (patch) | |
| tree | 31bc521ee99cf462e6041f04ac9b434a172f1340 /source/blender/windowmanager | |
| parent | 4f044f4ec1b682fb5b13f5aa62201314d855477b (diff) | |
Fix part 2 of [#31840] Quick Explode Bugs related to it's fade option.
This in fact had nothing to see with this operator or its fade option, reports were simply not shown when repeating an operator.
Carefully checked/tracked all calls to wm_operator_exec and WM_operator_repeat, and could not see any reason why this was this way, so I disabled this. Very easy to undo in case ugly problems arize (but I really do not expect any!).
Diffstat (limited to 'source/blender/windowmanager')
| -rw-r--r-- | source/blender/windowmanager/intern/wm_event_system.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/source/blender/windowmanager/intern/wm_event_system.c b/source/blender/windowmanager/intern/wm_event_system.c index 3d483c06419..e55f7404798 100644 --- a/source/blender/windowmanager/intern/wm_event_system.c +++ b/source/blender/windowmanager/intern/wm_event_system.c @@ -582,7 +582,10 @@ static int wm_operator_exec(bContext *C, wmOperator *op, int repeat) wm->op_undo_depth--; } - if (retval & (OPERATOR_FINISHED | OPERATOR_CANCELLED) && repeat == 0) + /* XXX Disabled the repeat check to address part 2 of #31840. + * Carefully checked all calls to wm_operator_exec and WM_operator_repeat, don't see any reason + * why this was needed, but worth to note it in case something turns bad. (mont29) */ + if (retval & (OPERATOR_FINISHED | OPERATOR_CANCELLED)/* && repeat == 0 */) wm_operator_reports(C, op, retval, FALSE); if (retval & OPERATOR_FINISHED) { |