[asterisk-commits] rmudgett: branch rmudgett/bridge_phase r393354 - /team/rmudgett/bridge_phase/...

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Mon Jul  1 15:40:18 2013
New Revision: 393354

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=393354
Log:
Defer a BUGBUG to if ever needed status.

Modified:
    team/rmudgett/bridge_phase/apps/app_agent_pool.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=393354&r1=393353&r2=393354
==============================================================================
--- team/rmudgett/bridge_phase/apps/app_agent_pool.c (original)
+++ team/rmudgett/bridge_phase/apps/app_agent_pool.c Mon Jul  1 15:40:18 2013
@@ -490,13 +490,6 @@
 enum agent_state {
 	/*! The agent is defined but an agent is not present. */
 	AGENT_STATE_LOGGED_OUT,
-/*
- * BUGBUG the login probation time should be only if it is needed.
- *
- * Need to determine if there are any local channels that can
- * optimize and wait until they actually do before leaving the
- * state.
- */
 	/*! Forced initial login wait to allow any local channel optimizations to happen. */
 	AGENT_STATE_PROBATION_WAIT,
 	/*! The agent is ready for a call. */
@@ -1160,6 +1153,14 @@
 	agent_lock(agent);
 	switch (agent->state) {
 	case AGENT_STATE_LOGGED_OUT:
+		/*!
+		 * \todo XXX the login probation time should be only if it is needed.
+		 *
+		 * Need to determine if there are any local channels that can
+		 * optimize and wait until they actually do before leaving the
+		 * AGENT_STATE_PROBATION_WAIT state.  For now, the blind
+		 * timer of LOGIN_WAIT_TIMEOUT_TIME will do.
+		 */
 		/*
 		 * Start the login probation timer.
 		 *