diff options
author | Richard Mudgett <rmudgett@digium.com> | 2014-04-15 18:01:47 +0000 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2014-04-15 18:01:47 +0000 |
commit | 45ade68cb48baad5f4f994fae6fe52211f379f70 (patch) | |
tree | 618aa6b37ae7714daa6090c4e7f2a9bc8ded1c0d /main | |
parent | d28af99e65c79f5bb1d336218f37de32313181db (diff) |
Remove unused RAII_VAR() declarations.
* Remove unused RAII_VAR() declarations. The compiler cannot catch these
because the cleanup function "references" the unused variable. Some
actually allocated and released resources that were never used.
* Fixed some whitespace issues in stasis_bridges.c.
........
Merged revisions 412399 from http://svn.asterisk.org/svn/asterisk/branches/12
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@412400 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main')
-rw-r--r-- | main/cdr.c | 6 | ||||
-rw-r--r-- | main/pbx.c | 2 | ||||
-rw-r--r-- | main/stasis_bridges.c | 36 | ||||
-rw-r--r-- | main/stasis_channels.c | 2 |
4 files changed, 18 insertions, 28 deletions
diff --git a/main/cdr.c b/main/cdr.c index 467969097..1828b0a4c 100644 --- a/main/cdr.c +++ b/main/cdr.c @@ -1085,7 +1085,6 @@ static struct ast_cdr *cdr_object_create_public_records(struct cdr_object *cdr) struct ast_var_t *it_var, *it_copy_var; struct ast_channel_snapshot *party_a; struct ast_channel_snapshot *party_b; - RAII_VAR(struct module_config *, mod_cfg, ao2_global_obj_ref(module_configs), ao2_cleanup); for (it_cdr = cdr; it_cdr; it_cdr = it_cdr->next) { struct ast_cdr *cdr_copy; @@ -1239,9 +1238,6 @@ static void cdr_object_set_disposition(struct cdr_object *cdr, int hangupcause) */ static void cdr_object_finalize(struct cdr_object *cdr) { - RAII_VAR(struct module_config *, mod_cfg, - ao2_global_obj_ref(module_configs), ao2_cleanup); - if (!ast_tvzero(cdr->end)) { return; } @@ -1644,8 +1640,6 @@ static enum ast_cdr_disposition dial_status_to_disposition(const char *dial_stat static int dial_state_process_dial_end(struct cdr_object *cdr, struct ast_channel_snapshot *caller, struct ast_channel_snapshot *peer, const char *dial_status) { - RAII_VAR(struct module_config *, mod_cfg, - ao2_global_obj_ref(module_configs), ao2_cleanup); struct ast_channel_snapshot *party_a; if (caller) { diff --git a/main/pbx.c b/main/pbx.c index b24bcd8c8..d16111746 100644 --- a/main/pbx.c +++ b/main/pbx.c @@ -4834,8 +4834,6 @@ static int pbx_extension_helper(struct ast_channel *c, struct ast_context *con, struct pbx_find_info q = { .stacklen = 0 }; /* the rest is reset in pbx_find_extension */ char passdata[EXT_DATA_SIZE]; int matching_action = (action == E_MATCH || action == E_CANMATCH || action == E_MATCHMORE); - RAII_VAR(struct ast_channel_snapshot *, snapshot, NULL, ao2_cleanup); - RAII_VAR(struct stasis_message *, msg, NULL, ao2_cleanup); ast_rdlock_contexts(); if (found) diff --git a/main/stasis_bridges.c b/main/stasis_bridges.c index eb020163d..1e0b36edf 100644 --- a/main/stasis_bridges.c +++ b/main/stasis_bridges.c @@ -346,11 +346,11 @@ static struct ast_json *ast_bridge_merge_message_to_json( return NULL; } - return ast_json_pack("{s: s, s: o, s: O, s: O}", - "type", "BridgeMerged", - "timestamp", ast_json_timeval(*stasis_message_timestamp(msg), NULL), - "bridge", json_bridge_to, - "bridge_from", json_bridge_from); + return ast_json_pack("{s: s, s: o, s: O, s: O}", + "type", "BridgeMerged", + "timestamp", ast_json_timeval(*stasis_message_timestamp(msg), NULL), + "bridge", json_bridge_to, + "bridge_from", json_bridge_from); } void ast_bridge_publish_merge(struct ast_bridge *to, struct ast_bridge *from) @@ -466,10 +466,10 @@ void ast_bridge_publish_leave(struct ast_bridge *bridge, struct ast_channel *cha } static struct ast_json *simple_bridge_channel_event( - const char *type, - struct ast_bridge_snapshot *bridge_snapshot, - struct ast_channel_snapshot *channel_snapshot, - const struct timeval *tv, + const char *type, + struct ast_bridge_snapshot *bridge_snapshot, + struct ast_channel_snapshot *channel_snapshot, + const struct timeval *tv, const struct stasis_message_sanitizer *sanitize) { RAII_VAR(struct ast_json *, json_bridge, @@ -481,31 +481,31 @@ static struct ast_json *simple_bridge_channel_event( return NULL; } - return ast_json_pack("{s: s, s: o, s: O, s: O}", - "type", type, - "timestamp", ast_json_timeval(*tv, NULL), - "bridge", json_bridge, - "channel", json_channel); + return ast_json_pack("{s: s, s: o, s: O, s: O}", + "type", type, + "timestamp", ast_json_timeval(*tv, NULL), + "bridge", json_bridge, + "channel", json_channel); } struct ast_json *ast_channel_entered_bridge_to_json( struct stasis_message *msg, const struct stasis_message_sanitizer *sanitize) { - struct ast_bridge_blob *obj = stasis_message_data(msg); + struct ast_bridge_blob *obj = stasis_message_data(msg); return simple_bridge_channel_event("ChannelEnteredBridge", obj->bridge, - obj->channel, stasis_message_timestamp(msg), sanitize); + obj->channel, stasis_message_timestamp(msg), sanitize); } struct ast_json *ast_channel_left_bridge_to_json( struct stasis_message *msg, const struct stasis_message_sanitizer *sanitize) { - struct ast_bridge_blob *obj = stasis_message_data(msg); + struct ast_bridge_blob *obj = stasis_message_data(msg); return simple_bridge_channel_event("ChannelLeftBridge", obj->bridge, - obj->channel, stasis_message_timestamp(msg), sanitize); + obj->channel, stasis_message_timestamp(msg), sanitize); } static struct ast_json *container_to_json_array(struct ao2_container *items, diff --git a/main/stasis_channels.c b/main/stasis_channels.c index 45ec53d7f..c75d17492 100644 --- a/main/stasis_channels.c +++ b/main/stasis_channels.c @@ -716,7 +716,6 @@ static struct ast_manager_event_blob *varset_to_ami(struct stasis_message *msg) static struct ast_manager_event_blob *agent_login_to_ami(struct stasis_message *msg) { RAII_VAR(struct ast_str *, channel_string, NULL, ast_free); - RAII_VAR(struct ast_str *, party_string, ast_str_create(256), ast_free); struct ast_channel_blob *obj = stasis_message_data(msg); const char *agent = ast_json_string_get(ast_json_object_get(obj->blob, "agent")); @@ -734,7 +733,6 @@ static struct ast_manager_event_blob *agent_login_to_ami(struct stasis_message * static struct ast_manager_event_blob *agent_logoff_to_ami(struct stasis_message *msg) { RAII_VAR(struct ast_str *, channel_string, NULL, ast_free); - RAII_VAR(struct ast_str *, party_string, ast_str_create(256), ast_free); struct ast_channel_blob *obj = stasis_message_data(msg); const char *agent = ast_json_string_get(ast_json_object_get(obj->blob, "agent")); long logintime = ast_json_integer_get(ast_json_object_get(obj->blob, "logintime")); |