[asterisk-commits] rmudgett: trunk r395088 - /trunk/main/bridging.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Jul 22 19:02:06 CDT 2013


Author: rmudgett
Date: Mon Jul 22 19:02:04 2013
New Revision: 395088

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=395088
Log:
Remove some BUGBUG notes that have been handled.

Modified:
    trunk/main/bridging.c

Modified: trunk/main/bridging.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/bridging.c?view=diff&rev=395088&r1=395087&r2=395088
==============================================================================
--- trunk/main/bridging.c (original)
+++ trunk/main/bridging.c Mon Jul 22 19:02:04 2013
@@ -4899,21 +4899,6 @@
 		return 0;
 	}
 
-/*
- * BUGBUG Need to take into account frame hooks on the unreal/local channels optimization.
- *
- * Frame hooks on the unreal/local channels may inhibit optimization here if they cannot
- * be moved to an appropriate peer channel.
- *
- * caller -- Bridge -- L;1 -- L;2 -- Bridge -- agent
- * Any frame hooks on L;1 need to be moved to agent channel.
- * Any frame hooks on L;2 need to be moved to caller channel.
- *
- * Moving the frame hooks may cause the hooks to miss a frame if
- * the destination channel has already read a frame but cannot
- * write it into the bridge yet because optimization has the
- * bridge locked.
- */
 	other = ast_bridge_channel_peer(src_bridge_channel);
 	if (other && other->state == AST_BRIDGE_CHANNEL_STATE_WAIT) {
 		ast_verb(3, "Move-swap optimizing %s <-- %s.\n",
@@ -5028,7 +5013,6 @@
 		return 0;
 	}
 
-/* BUGBUG Frame hooks on the unreal/local channels need to inhibit optimization here. */
 	ast_verb(3, "Merge optimizing %s -- %s out.\n",
 		ast_channel_name(chan_bridge_channel->chan),
 		ast_channel_name(peer_bridge_channel->chan));




More information about the asterisk-commits mailing list