[asterisk-commits] mmichelson: branch mmichelson/digit_manipulation r197193 - /team/mmichelson/d...

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Wed May 27 14:08:50 2009
New Revision: 197193

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=197193
Log:
Fix comments a bit more.


Modified:
    team/mmichelson/digit_manipulation/channels/chan_sip.c

Modified: team/mmichelson/digit_manipulation/channels/chan_sip.c
URL: http://svn.asterisk.org/svn-view/asterisk/team/mmichelson/digit_manipulation/channels/chan_sip.c?view=diff&rev=197193&r1=197192&r2=197193
==============================================================================
--- team/mmichelson/digit_manipulation/channels/chan_sip.c (original)
+++ team/mmichelson/digit_manipulation/channels/chan_sip.c Wed May 27 14:08:50 2009
@@ -20453,10 +20453,13 @@
 			ast_channel_unlock(targetcall_pvt->owner);
 		}
 
-		/* Tell each of the other channels to whom they are now connected */
-		/* No need to lock target.chan1 here since it was previously locked in get_sip_pvt_byid_locked */
+		/* By forcing the masquerade, we know that target.chan1 and target.chan2 are bridged. We then
+		 * can queue connected line updates where they need to go.
+		 *
+		 * No need to lock target.chan1 here since it was previously locked in get_sip_pvt_byid_locked
+		 */
 		if (target.chan1->masq) {
-		/* If the channel thread already did the masquerade, then we don't need to do anything */
+			/* If the channel thread already did the masquerade, then we don't need to do anything */
 			ast_do_masquerade(target.chan1);
 		}
 		if (target.chan2) {