diff options
author | Luigi Rizzo <rizzo@icir.org> | 2006-05-10 10:10:03 +0000 |
---|---|---|
committer | Luigi Rizzo <rizzo@icir.org> | 2006-05-10 10:10:03 +0000 |
commit | 298eec4d28c693bcdd8e4b9ebd8120bcbf551ac7 (patch) | |
tree | 8f9600f617dfa23c3319709b79048e07be982a95 /apps/app_voicemail.c | |
parent | 66ed134473faeb5bf3bac6e00fe928d49e60f62f (diff) |
use 'switch' insteaf of multiple 'if'
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@26345 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'apps/app_voicemail.c')
-rw-r--r-- | apps/app_voicemail.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c index 063eff45b..884cbdd88 100644 --- a/apps/app_voicemail.c +++ b/apps/app_voicemail.c @@ -6192,7 +6192,7 @@ static int load_config(void) AST_LIST_LOCK(&users); while ((cur = AST_LIST_REMOVE_HEAD(&users, list))) { - ast_set_flag(cur, VM_ALLOCED); + ast_set_flag(cur, VM_ALLOCED); free_user(cur); } @@ -6752,16 +6752,17 @@ static int advanced_options(struct ast_channel *chan, struct ast_vm_user *vmu, s if (!strncasecmp("macro",context,5)) /* Macro names in contexts are useless for our needs */ context = ast_variable_retrieve(msg_cfg, "message","macrocontext"); - if (option == 3) { - + switch (option) { + case 3: if (!res) res = play_message_datetime(chan, vmu, origtime, filename); if (!res) res = play_message_callerid(chan, vms, cid, context, 0); res = 't'; + break; - } else if (option == 2) { /* Call back */ + case 2: /* Call back */ if (!ast_strlen_zero(cid)) { ast_callerid_parse(cid, &name, &num); @@ -6843,9 +6844,9 @@ static int advanced_options(struct ast_channel *chan, struct ast_vm_user *vmu, s res = -1; } } + break; - } - else if (option == 1) { /* Reply */ + case 1: /* Reply */ /* Send reply directly to sender */ if (!ast_strlen_zero(cid)) { ast_callerid_parse(cid, &name, &num); @@ -6881,6 +6882,7 @@ static int advanced_options(struct ast_channel *chan, struct ast_vm_user *vmu, s } res = 0; } + break; } ast_config_destroy(msg_cfg); |