Made playback operators use exec() callback instead of invoke(), so that these can...
authorJoshua Leung <aligorith@gmail.com>
Tue, 20 Apr 2010 02:39:07 +0000 (02:39 +0000)
committerJoshua Leung <aligorith@gmail.com>
Tue, 20 Apr 2010 02:39:07 +0000 (02:39 +0000)
Note that this is not the patch by dfelinto on the mailing list, since that fix would cause compiler warnings. Also, the invoke() (with the extra wmEvent* arg) is superfluous here, so there shouldn't be any problems with making this exec() only instead.

source/blender/editors/screen/screen_ops.c

index db67091..2449977 100644 (file)
@@ -2561,7 +2561,7 @@ int ED_screen_animation_play(bContext *C, int sync, int mode)
        return OPERATOR_FINISHED;
 }
 
-static int screen_animation_play_invoke(bContext *C, wmOperator *op, wmEvent *event)
+static int screen_animation_play_exec(bContext *C, wmOperator *op)
 {
        int mode= (RNA_boolean_get(op->ptr, "reverse")) ? -1 : 1;
        int sync= -1;
@@ -2580,7 +2580,7 @@ static void SCREEN_OT_animation_play(wmOperatorType *ot)
        ot->idname= "SCREEN_OT_animation_play";
        
        /* api callbacks */
-       ot->invoke= screen_animation_play_invoke;
+       ot->exec= screen_animation_play_exec;
        
        ot->poll= ED_operator_screenactive;
        
@@ -2588,7 +2588,7 @@ static void SCREEN_OT_animation_play(wmOperatorType *ot)
        RNA_def_boolean(ot->srna, "sync", 0, "Sync", "Drop frames to maintain framerate");
 }
 
-static int screen_animation_cancel(bContext *C, wmOperator *op, wmEvent *event)
+static int screen_animation_cancel_exec(bContext *C, wmOperator *op)
 {
        bScreen *screen= CTX_wm_screen(C);
        
@@ -2617,7 +2617,7 @@ static void SCREEN_OT_animation_cancel(wmOperatorType *ot)
        ot->idname= "SCREEN_OT_animation_cancel";
        
        /* api callbacks */
-       ot->invoke= screen_animation_cancel;
+       ot->exec= screen_animation_cancel_exec;
        
        ot->poll= ED_operator_screenactive;
 }