[svn-commits] oej: branch oej/danishtimestamps-1.8 r417750 - /team/oej/danishtimestamps-1.8...

SVN commits to the Digium repositories svn-commits at lists.digium.com
Wed Jul 2 05:44:26 CDT 2014


Author: oej
Date: Wed Jul  2 05:44:24 2014
New Revision: 417750

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=417750
Log:
Not ready to delete yet, but ready to disable.

If anyone can come up with a reason why this piece of disabled code is good,
please, please, tell me NOW!

Modified:
    team/oej/danishtimestamps-1.8/res/res_rtp_asterisk.c

Modified: team/oej/danishtimestamps-1.8/res/res_rtp_asterisk.c
URL: http://svnview.digium.com/svn/asterisk/team/oej/danishtimestamps-1.8/res/res_rtp_asterisk.c?view=diff&rev=417750&r1=417749&r2=417750
==============================================================================
--- team/oej/danishtimestamps-1.8/res/res_rtp_asterisk.c (original)
+++ team/oej/danishtimestamps-1.8/res/res_rtp_asterisk.c Wed Jul  2 05:44:24 2014
@@ -1238,9 +1238,14 @@
 		rtp->lastdigitts = rtp->lastts;
 	}
 
+#ifdef MAYBE_BROKEN_CODE
+	/* This means that we strictly follow the timestamps of the incoming stream, which may
+	   be unrelated to our send stream. I think it's a bad idea. /OEJ 
+         */
 	if (ast_test_flag(frame, AST_FRFLAG_HAS_TIMING_INFO)) {
 		rtp->lastts = frame->ts * rate;
 	}
+#endif
 
 	ast_rtp_instance_get_remote_address(instance, &remote_address);
 




More information about the svn-commits mailing list