[asterisk-commits] rmudgett: branch rmudgett/bridge_phase r393614 - in /team/rmudgett/bridge_pha...

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Wed Jul  3 17:54:20 2013
New Revision: 393614

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=393614
Log:
Cleanup some BUGBUG notes for agents.

Modified:
    team/rmudgett/bridge_phase/apps/app_agent_pool.c
    team/rmudgett/bridge_phase/main/config_options.c

Modified: team/rmudgett/bridge_phase/apps/app_agent_pool.c
URL: http://svnview.digium.com/svn/asterisk/team/rmudgett/bridge_phase/apps/app_agent_pool.c?view=diff&rev=393614&r1=393613&r2=393614
==============================================================================
--- team/rmudgett/bridge_phase/apps/app_agent_pool.c (original)
+++ team/rmudgett/bridge_phase/apps/app_agent_pool.c Wed Jul  3 17:54:20 2013
@@ -413,15 +413,6 @@
 	.types = ACO_TYPES(&general_type, &agent_type),
 };
 
-/*
- * Move this note into config framework and create an issue.
- *
- * BUGBUG must fix config framework loading of multiple files.
- *
- * A reload with multiple files must reload all files if any
- * file has been touched.
- */
-
 static AO2_GLOBAL_OBJ_STATIC(cfg_handle);
 
 static void agents_cfg_destructor(void *vdoomed)
@@ -1955,15 +1946,6 @@
 		ast_getformatname(ast_channel_writeformat(chan)));
 	send_agent_login(chan, agent->username);
 
-/* BUGBUG Debug force setting of dtmf features datastore options until CHANNEL(dtmf-featurs) is available. */
-	{
-		struct ast_flags flags = { AST_FEATURE_DTMF_MASK };
-
-		ast_channel_lock(chan);
-		ast_bridge_features_ds_set(chan, &flags);
-		ast_channel_unlock(chan);
-	}
-
 	agent_run(agent, chan);
 	return -1;
 }

Modified: team/rmudgett/bridge_phase/main/config_options.c
URL: http://svnview.digium.com/svn/asterisk/team/rmudgett/bridge_phase/main/config_options.c?view=diff&rev=393614&r1=393613&r2=393614
==============================================================================
--- team/rmudgett/bridge_phase/main/config_options.c (original)
+++ team/rmudgett/bridge_phase/main/config_options.c Wed Jul  3 17:54:20 2013
@@ -598,6 +598,12 @@
 		return ACO_PROCESS_ERROR;
 	}
 
+/*
+ * BUGBUG must fix config framework loading of multiple files.
+ *
+ * A reload with multiple files must reload all files if any
+ * file has been touched.
+ */
 	while (res != ACO_PROCESS_ERROR && (file = info->files[x++])) {
 		const char *filename = file->filename;
 try_alias: