Merge in ast_strftime branch, which changes timestamps to be accurate to the microsec...
[asterisk/asterisk.git] / funcs / func_strings.c
index af59944..c44c499 100644 (file)
@@ -22,6 +22,7 @@
  *
  * \author Tilghman Lesher
  * \author Anothony Minessale II 
+ * \ingroup functions
  */
 
 #include "asterisk.h"
@@ -47,7 +48,7 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
 static int function_fieldqty(struct ast_channel *chan, const char *cmd,
                             char *parse, char *buf, size_t len)
 {
-       char *varval;
+       char *varsubst, varval[8192] = "", *varval2 = varval;
        int fieldcount = 0;
        AST_DECLARE_APP_ARGS(args,
                             AST_APP_ARG(varname);
@@ -56,7 +57,6 @@ static int function_fieldqty(struct ast_channel *chan, const char *cmd,
 
        AST_STANDARD_APP_ARGS(args, parse);
        if (args.delim) {
-               pbx_retrieve_variable(chan, args.varname, &varval, buf, len, NULL);
                if (args.delim[0] == '\\') {
                        if (args.delim[1] == 'n')
                                ast_copy_string(args.delim, "\n", 2);
@@ -67,8 +67,16 @@ static int function_fieldqty(struct ast_channel *chan, const char *cmd,
                        else
                                ast_copy_string(args.delim, "-", 2);
                }
-               while (strsep(&varval, args.delim))
-                       fieldcount++;
+               varsubst = alloca(strlen(args.varname) + 4);
+
+               sprintf(varsubst, "${%s}", args.varname);
+               pbx_substitute_variables_helper(chan, varsubst, varval, sizeof(varval) - 1);
+               if (ast_strlen_zero(varval2))
+                       fieldcount = 0;
+               else {
+                       while (strsep(&varval2, args.delim))
+                               fieldcount++;
+               }
        } else {
                fieldcount = 1;
        }
@@ -138,8 +146,7 @@ static int regex(struct ast_channel *chan, const char *cmd, char *parse, char *b
        if ((*args.str == ' ') || (*args.str == '\t'))
                args.str++;
 
-       if (option_debug)
-               ast_log(LOG_DEBUG, "FUNCTION REGEX (%s)(%s)\n", args.reg, args.str);
+       ast_debug(1, "FUNCTION REGEX (%s)(%s)\n", args.reg, args.str);
 
        if ((errcode = regcomp(&regexbuf, args.reg, REG_EXTENDED | REG_NOSUB))) {
                regerror(errcode, &regexbuf, buf, len);
@@ -183,7 +190,7 @@ static void clearvar_prefix(struct ast_channel *chan, const char *prefix)
        AST_LIST_TRAVERSE_SAFE_BEGIN(&chan->varshead, var, entries) {
                if (strncasecmp(prefix, ast_var_name(var), len) == 0) {
                        AST_LIST_REMOVE_CURRENT(&chan->varshead, entries);
-                       free(var);
+                       ast_free(var);
                }
        }
        AST_LIST_TRAVERSE_SAFE_END
@@ -229,8 +236,7 @@ static int array(struct ast_channel *chan, const char *cmd, char *var,
         * want them to be surprised by the result.  Hence, we prefer commas as the
         * delimiter, but we'll fall back to vertical bars if commas aren't found.
         */
-       if (option_debug)
-               ast_log(LOG_DEBUG, "array (%s=%s)\n", var, value2);
+       ast_debug(1, "array (%s=%s)\n", var, value2);
        if (strchr(var, ','))
                AST_NONSTANDARD_APP_ARGS(arg1, var, ',');
        else
@@ -242,8 +248,7 @@ static int array(struct ast_channel *chan, const char *cmd, char *var,
                AST_STANDARD_APP_ARGS(arg2, value2);
 
        for (i = 0; i < arg1.argc; i++) {
-               if (option_debug)
-                       ast_log(LOG_DEBUG, "array set value (%s=%s)\n", arg1.var[i],
+               ast_debug(1, "array set value (%s=%s)\n", arg1.var[i],
                                arg2.val[i]);
                if (i < arg2.argc) {
                        if (ishash) {
@@ -454,7 +459,7 @@ static int acf_sprintf(struct ast_channel *chan, const char *cmd, char *data, ch
                                formatbuf[&arg.format[i] - formatstart + 1] = '\0';
 
                                /* Convert the argument into the required type */
-                               if (sscanf(arg.var[argcount++], "%i", &tmpi) != 1) {
+                               if (sscanf(arg.var[argcount++], "%d", &tmpi) != 1) {
                                        ast_log(LOG_ERROR, "Argument '%s' is not an integer number for format '%s'\n", arg.var[argcount - 1], formatbuf);
                                        goto sprintf_fail;
                                }
@@ -594,20 +599,20 @@ static int acf_strftime(struct ast_channel *chan, const char *cmd, char *parse,
                             AST_APP_ARG(timezone);
                             AST_APP_ARG(format);
        );
-       time_t epochi;
-       struct tm tm;
+       struct timeval tv;
+       struct ast_tm tm;
 
        buf[0] = '\0';
 
        AST_STANDARD_APP_ARGS(args, parse);
 
-       ast_get_time_t(args.epoch, &epochi, time(NULL), NULL);
-       ast_localtime(&epochi, &tm, args.timezone);
+       ast_get_timeval(args.epoch, &tv, ast_tvnow(), NULL);
+       ast_localtime(&tv, &tm, args.timezone);
 
        if (!args.format)
                args.format = "%c";
 
-       if (!strftime(buf, len, args.format, &tm))
+       if (ast_strftime(buf, len, args.format, &tm) <= 0)
                ast_log(LOG_WARNING, "C function strftime() output nothing?!!\n");
 
        buf[len - 1] = '\0';
@@ -619,6 +624,14 @@ static struct ast_custom_function strftime_function = {
        .name = "STRFTIME",
        .synopsis = "Returns the current date/time in a specified format.",
        .syntax = "STRFTIME([<epoch>][|[timezone][|format]])",
+       .desc =
+"STRFTIME sports all of the same formats as the underlying C function\n"
+"strftime(3) - see the man page for details.  It also supports the\n"
+"following format:\n"
+" %[n]q - fractions of a second, with leading zeroes.  For example, %3q will\n"
+"         give milliseconds and %1q will give tenths of a second.  The default\n"
+"         is to output milliseconds (n=3).  The common case is to use it in\n"
+"         combination with %S, as in \"%S.%3q\".\n",
        .read = acf_strftime,
 };
 
@@ -630,9 +643,10 @@ static int acf_strptime(struct ast_channel *chan, const char *cmd, char *data,
                             AST_APP_ARG(timezone);
                             AST_APP_ARG(format);
        );
-       struct tm time;
-
-       memset(&time, 0, sizeof(struct tm));
+       union {
+               struct ast_tm atm;
+               struct tm time;
+       } t = { { 0, }, };
 
        buf[0] = '\0';
 
@@ -650,10 +664,10 @@ static int acf_strptime(struct ast_channel *chan, const char *cmd, char *data,
                return -1;
        }
 
-       if (!strptime(args.timestring, args.format, &time)) {
+       if (!strptime(args.timestring, args.format, &t.time)) {
                ast_log(LOG_WARNING, "C function strptime() output nothing?!!\n");
        } else {
-               snprintf(buf, len, "%d", (int) ast_mktime(&time, args.timezone));
+               snprintf(buf, len, "%d", (int) ast_mktime(&t.atm, args.timezone));
        }
 
        return 0;
@@ -677,7 +691,7 @@ static struct ast_custom_function strptime_function = {
 static int function_eval(struct ast_channel *chan, const char *cmd, char *data,
                         char *buf, size_t len)
 {
-       buf[0] = '\0';
+       memset(buf, 0, len);
 
        if (ast_strlen_zero(data)) {
                ast_log(LOG_WARNING, "EVAL requires an argument: EVAL(<string>)\n");