diff options
author | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-06-04 17:32:41 +0000 |
---|---|---|
committer | file <file@f38db490-d61c-443f-a65b-d21fe96a405b> | 2007-06-04 17:32:41 +0000 |
commit | 569a88fb54f288946cc4f16aee2c049fe2bee3b1 (patch) | |
tree | 42de27a6c61b7431cc962a96f56211fe6d8322b3 /apps/app_directed_pickup.c | |
parent | cd81d42594299b5ddf6646c0607c85e741a67f5f (diff) |
Minor clean up. Constify a few variables and use ast_strlen_zero in a few places.
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@67063 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'apps/app_directed_pickup.c')
-rw-r--r-- | apps/app_directed_pickup.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/apps/app_directed_pickup.c b/apps/app_directed_pickup.c index 9aba195ec..60741fcb6 100644 --- a/apps/app_directed_pickup.c +++ b/apps/app_directed_pickup.c @@ -90,7 +90,7 @@ static int can_pickup(struct ast_channel *chan) } /* Attempt to pick up specified extension with context */ -static int pickup_by_exten(struct ast_channel *chan, char *exten, char *context) +static int pickup_by_exten(struct ast_channel *chan, const char *exten, const char *context) { int res = -1; struct ast_channel *target = NULL; @@ -110,7 +110,7 @@ static int pickup_by_exten(struct ast_channel *chan, char *exten, char *context) } /* Attempt to pick up specified mark */ -static int pickup_by_mark(struct ast_channel *chan, char *mark) +static int pickup_by_mark(struct ast_channel *chan, const char *mark) { int res = -1; const char *tmp = NULL; @@ -149,11 +149,11 @@ static int pickup_exec(struct ast_channel *chan, void *data) while (!ast_strlen_zero(tmp) && (exten = strsep(&tmp, "&"))) { if ((context = strchr(exten, '@'))) *context++ = '\0'; - if (context && !strcasecmp(context, PICKUPMARK)) { + if (!ast_strlen_zero(context) && !strcasecmp(context, PICKUPMARK)) { if (!pickup_by_mark(chan, exten)) break; } else { - if (!pickup_by_exten(chan, exten, context ? context : chan->context)) + if (!pickup_by_exten(chan, exten, !ast_strlen_zero(context) ? context : chan->context)) break; } ast_log(LOG_NOTICE, "No target channel found for %s.\n", exten); |