add 'restart' character to ast_control_playback function and ControlPlayback() applic...
authorKevin P. Fleming <kpfleming@digium.com>
Mon, 22 Aug 2005 19:06:42 +0000 (19:06 +0000)
committerKevin P. Fleming <kpfleming@digium.com>
Mon, 22 Aug 2005 19:06:42 +0000 (19:06 +0000)
rework app_controlplayback.c to take into account all Asterisk 1.2 coding style requirements

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@6352 65c4cc65-6c06-0410-ace0-fbb531ad65f3

app.c
apps/app_controlplayback.c
apps/app_voicemail.c
include/asterisk/app.h

diff --git a/app.c b/app.c
index e571694..f3a84c3 100755 (executable)
--- a/app.c
+++ b/app.c
@@ -406,7 +406,10 @@ int ast_linear_stream(struct ast_channel *chan, const char *filename, int fd, in
        return res;
 }
 
-int ast_control_streamfile(struct ast_channel *chan, const char *file, const char *fwd, const char *rev, const char *stop, const char *pause, int skipms) 
+int ast_control_streamfile(struct ast_channel *chan, const char *file,
+                          const char *fwd, const char *rev,
+                          const char *stop, const char *pause,
+                          const char *restart, int skipms) 
 {
        long elapsed = 0,last_elapsed =0;
        char *breaks=NULL;
@@ -418,12 +421,18 @@ int ast_control_streamfile(struct ast_channel *chan, const char *file, const cha
                blen += strlen(stop);
        if (pause)
                blen += strlen(pause);
+       if (restart)
+               blen += strlen(restart);
 
        if (blen > 2) {
                breaks = alloca(blen + 1);
                breaks[0] = '\0';
-               strcat(breaks, stop);
-               strcat(breaks, pause);
+               if (stop)
+                       strcat(breaks, stop);
+               if (pause)
+                       strcat(breaks, pause);
+               if (restart)
+                       strcat(breaks, restart);
        }
        if (chan->_state != AST_STATE_UP)
                res = ast_answer(chan);
@@ -466,6 +475,13 @@ int ast_control_streamfile(struct ast_channel *chan, const char *file, const cha
                if (res < 1)
                        break;
 
+               /* We go at next loop if we got the restart char */
+               if (restart && strchr(restart, res)) {
+                       ast_log(LOG_DEBUG, "we'll restart the stream here at next loop\n");
+                       elapsed=0; /* To make sure the next stream will start at beginning */
+                       continue;
+               }
+
                if (pause != NULL && strchr(pause, res)) {
                        elapsed = ast_tvdiff_ms(ast_tvnow(), started) + last_elapsed;
                        for(;;) {
index 30b5c90..da29c7f 100755 (executable)
@@ -28,20 +28,21 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 #include "asterisk/translate.h"
 #include "asterisk/utils.h"
 
-static char *tdesc = "Control Playback Application";
+static const char *tdesc = "Control Playback Application";
 
-static char *app = "ControlPlayback";
+static const char *app = "ControlPlayback";
 
-static char *synopsis = "Play a file with fast forward and rewind";
+static const char *synopsis = "Play a file with fast forward and rewind";
 
-static char *descrip = 
-"ControlPlayback(filename[|skipms[|ffchar[|rewchar[|stopchar[|pausechr]]]]]):\n"
+static const char *descrip = 
+"ControlPlayback(filename[|skipms[|ffchar[|rewchar[|stopchar[|pausechar[|restartchar]]]]]]):\n"
 "  Plays  back  a  given  filename (do not put extension). Options may also\n"
 "  be included following a pipe symbol.  You can use * and # to rewind and\n"
 "  fast forward the playback specified. If 'stopchar' is added the file will\n"
-"  terminate playback when 'stopchar' is pressed. Returns -1 if the channel\n"
+"  terminate playback when 'stopchar' is pressed. If 'restartchar' is added, the file\n"
+"  will restart when 'restartchar' is pressed. Returns -1 if the channel\n"
 "  was hung up. if the file does not exist jumps to n+101 if it present.\n\n"
-"  Example:  exten => 1234,1,ControlPlayback(file|4000|*|#|1|0)\n\n";
+"  Example:  exten => 1234,1,ControlPlayback(file|4000|*|#|1|0|5)\n\n";
 
 STANDARD_LOCAL_USER;
 
@@ -49,7 +50,7 @@ LOCAL_USER_DECL;
 
 static int is_on_phonepad(char key)
 {
-       return (key == 35 || key == 42 || (key >= 48 && key <= 57)) ? 1 : 0;
+       return key == 35 || key == 42 || (key >= 48 && key <= 57);
 }
 
 static int controlplayback_exec(struct ast_channel *chan, void *data)
@@ -57,72 +58,71 @@ static int controlplayback_exec(struct ast_channel *chan, void *data)
        int res = 0;
        int skipms = 0;
        struct localuser *u;
-       char tmp[256];
-       char *skip = NULL, *fwd = NULL, *rev = NULL, *stop = NULL, *pause = NULL, *file = NULL;
-       
+       char *tmp;
+       int argc;
+       char *argv[7];
+       enum arg_ids {
+               arg_file = 0,
+               arg_skip = 1,
+               arg_fwd = 2,
+               arg_rev = 3,
+               arg_stop = 4,
+               arg_pause = 5,
+               arg_restart = 6,
+       };
 
        if (!data || ast_strlen_zero((char *)data)) {
                ast_log(LOG_WARNING, "ControlPlayback requires an argument (filename)\n");
                return -1;
        }
 
-       strncpy(tmp, (char *)data, sizeof(tmp)-1);
-       file = tmp;
-
-       if ((skip=strchr(tmp,'|'))) {
-               *skip++ = '\0';
-               fwd=strchr(skip,'|');
-               if (fwd) {
-                       *fwd++ = '\0';
-                       rev = strchr(fwd,'|');
-                       if (rev) {
-                               *rev++ = '\0';
-                               stop = strchr(rev,'|');
-                               if (stop) {
-                                       *stop++ = '\0';
-                                       pause = strchr(stop,'|');
-                                       if (pause) {
-                                               *pause++ = '\0';
-                                       }
-                               }
-                       }
-               }
+       tmp = ast_strdupa(data);
+       memset(argv, 0, sizeof(argv));
+
+       argc = ast_separate_app_args(tmp, '|', argv, sizeof(argv) / sizeof(argv[0]));
+
+       if (argc < 1) {
+               ast_log(LOG_WARNING, "ControlPlayback requires an argument (filename)\n");
+               return -1;
        }
 
-       skipms = skip ? atoi(skip) : 3000;
+       skipms = argv[arg_skip] ? atoi(argv[arg_skip]) : 3000;
        if (!skipms)
                skipms = 3000;
 
-       if (!fwd || !is_on_phonepad(*fwd))
-               fwd = "#";
-       if (!rev || !is_on_phonepad(*rev))
-               rev = "*";
-       if (stop && !is_on_phonepad(*stop))
-               stop = NULL;
-       if (pause && !is_on_phonepad(*pause))
-               pause = NULL;
+       if (!argv[arg_fwd] || !is_on_phonepad(*argv[arg_fwd]))
+               argv[arg_fwd] = "#";
+       if (!argv[arg_rev] || !is_on_phonepad(*argv[arg_rev]))
+               argv[arg_rev] = "*";
+       if (argv[arg_stop] && !is_on_phonepad(*argv[arg_stop]))
+               argv[arg_stop] = NULL;
+       if (argv[arg_pause] && !is_on_phonepad(*argv[arg_pause]))
+               argv[arg_pause] = NULL;
+       if (argv[arg_restart] && !is_on_phonepad(*argv[arg_restart]))
+               argv[arg_restart] = NULL;
 
        LOCAL_USER_ADD(u);
 
-       res = ast_control_streamfile(chan, file, fwd, rev, stop, pause, skipms);
+       res = ast_control_streamfile(chan, argv[arg_file], argv[arg_fwd], argv[arg_rev], argv[arg_stop], argv[arg_pause], argv[arg_restart], skipms);
 
        LOCAL_USER_REMOVE(u);
        
        /* If we stopped on one of our stop keys, return 0  */
-       if(stop && strchr(stop, res)) 
+       if (argv[arg_stop] && strchr(argv[arg_stop], res)) 
                res = 0;
 
-       if(res < 0) {
-               if (ast_exists_extension(chan, chan->context, chan->exten, chan->priority + 101, chan->cid.cid_num))
-            chan->priority+=100;
-               res = 0;
+       if (res < 0) {
+               if (ast_goto_if_exists(chan, chan->context, chan->exten, chan->priority + 101))
+                       res = 0;
        }
+
        return res;
 }
 
 int unload_module(void)
 {
        STANDARD_HANGUP_LOCALUSERS;
+
        return ast_unregister_application(app);
 }
 
@@ -133,12 +133,13 @@ int load_module(void)
 
 char *description(void)
 {
-       return tdesc;
+       return (char *) tdesc;
 }
 
 int usecount(void)
 {
        int res;
+
        STANDARD_USECOUNT(res);
        return res;
 }
index dbe0639..dabadcd 100755 (executable)
@@ -3255,7 +3255,7 @@ static int wait_file2(struct ast_channel *chan, struct vm_state *vms, char *file
 
 static int wait_file(struct ast_channel *chan, struct vm_state *vms, char *file) 
 {
-       return ast_control_streamfile(chan, file, "#", "*", "1456789", "0", skipms);
+       return ast_control_streamfile(chan, file, "#", "*", "1456789", "0", "2", skipms);
 }
 
 static int play_message_category(struct ast_channel *chan, char *category)
index 4ab7ef2..79fee29 100755 (executable)
@@ -124,8 +124,8 @@ int ast_dtmf_stream(struct ast_channel *chan, struct ast_channel *peer, char *di
 /*! Stream a filename (or file descriptor) as a generator. */
 int ast_linear_stream(struct ast_channel *chan, const char *filename, int fd, int allowoverride);
 
-/*! Stream a file with fast forward, pause, reverse. */
-int ast_control_streamfile(struct ast_channel *chan, const char *file, const char *fwd, const char *rev, const char *stop, const char *pause, int skipms);
+/*! Stream a file with fast forward, pause, reverse, restart. */
+int ast_control_streamfile(struct ast_channel *chan, const char *file, const char *fwd, const char *rev, const char *stop, const char *pause, const char *restart, int skipms);
 
 /*! Play a stream and wait for a digit, returning the digit that was pressed */
 int ast_play_and_wait(struct ast_channel *chan, const char *fn);