[Sumover-dev] [svn commit] r4355 - vic/branches/cc/rtp

sumover-dev at cs.ucl.ac.uk sumover-dev at cs.ucl.ac.uk
Thu Jan 22 16:02:34 GMT 2009


Author: soohyunc
Date: Thu Jan 22 16:02:33 2009
New Revision: 4355

Modified:
   vic/branches/cc/rtp/session.cpp

Log:
o	change RTP data sender's routines and receiver's routine 
	in parse_xr_records() method - in this way, it is more straight forward.



Modified: vic/branches/cc/rtp/session.cpp
==============================================================================
--- vic/branches/cc/rtp/session.cpp	(original)
+++ vic/branches/cc/rtp/session.cpp	Thu Jan 22 16:02:33 2009
@@ -1173,8 +1173,27 @@
 	u_int16_t end	= ntohs(xr->end_seq);
 	u_int16_t chunk	= ntohs(xr->chunk);
 
+	// i am an RTP data sender, so do the sender stuffs
+	if (am_i_sender()) {
+		// parse AckVec and ts echo from XR report block
+		if (flags == XR_BT_1) {
+			ackvec_ = ntohl(xr->chunk);
+
+			// this XR conveys AckVec from data receiver
+			tfwc_sndr_recv(flags, ackvec_, 0);
+		} 
+		else if (flags == XR_BT_3) {
+			ts_echo_ = ntohl(xr->chunk);
+
+			// this XR conveys ts echo
+			tfwc_sndr_recv(flags, 0, ts_echo_);
+		}
+
+		// we need to call Transmitter::output(pb) to make Ack driven
+		cc_output();
+	}
 	// i am an RTP data receiver, so do the receiver stuffs
-	if (!am_i_sender()) {
+	else {
 		// parse seqno, ackofack, and timestamp from XR report block
 		if(flags == XR_BT_1) {
 			// this is XR conveys seqno and ackofack from data sender
@@ -1191,25 +1210,6 @@
 		ch_[0].send_ackv();
 		//ch_[0].send_ts_echo();
 	}
-	// i am an RTP data sender, so do the sender stuffs
-	else {
-		// parse ackvec and timestamp echo from XR report block
-		if(flags == XR_BT_1) {
-			ackvec_ = ntohl(xr->chunk);
-
-			// this XR conveys ackvec from data receiver, hence parse it
-			tfwc_sndr_recv(flags, ackvec_, 0);
-		}
-		else if(flags == XR_BT_3) {
-			ts_echo_ = ntohl(xr->chunk);
-
-			// this XR conveys ts echo, hence parse it
-			tfwc_sndr_recv(flags, 0, ts_echo_);
-		}
-
-		// we need to call Transmitter::output(pb) here (make Ack driven)
-		cc_output();
-	}
 }
 
 void SessionManager::cc_output() 



More information about the Sumover-dev mailing list