Author: derek Date: Sun Nov 5 22:12:57 2006 New Revision: 156 Modified: sofaswitch/trunk/mcu/participant.cxx sofaswitch/trunk/mcu/participant.h Log: tidy up the handling of one method, and make the name/comment more meaningful Modified: sofaswitch/trunk/mcu/participant.cxx ============================================================================== --- sofaswitch/trunk/mcu/participant.cxx (original) +++ sofaswitch/trunk/mcu/participant.cxx Sun Nov 5 22:12:57 2006 @@ -114,18 +114,13 @@ { PTRACE(3, "OnRelease thread started for " << *this); - DestroyNeighboursLink(); + DestroyReverseLink(); OnReleased(); PTRACE(3, "Participant OnRelease thread completed for " << *this); // Dereference on the way out of the thread SafeDereference(); -} - -void Participant::DestroyNeighboursLink() -{ - dstMemberPtr->DestroyThisParticipant(matchingKey); } Modified: sofaswitch/trunk/mcu/participant.h ============================================================================== --- sofaswitch/trunk/mcu/participant.h (original) +++ sofaswitch/trunk/mcu/participant.h Sun Nov 5 22:12:57 2006 @@ -253,9 +253,13 @@ */ virtual void OnReleased(); - /**Destroy the link from us to the other. This causees the other to - quietly destroy its link to us, but not tell us it has died */ - void DestroyNeighboursLink(); + /**Destroy the reverse link. That is, the link from dstMemberId to + srcMemberId(this). + + Remember, this participant instance is from srcMemberId(this) to + dstMemberId; + */ + void DestroyReverseLink() { dstMemberPtr->DestroyThisParticipant(matchingKey); } /**link to the manager, who knows everything about everything */