[asterisk-commits] r411702 - svn:log

[SVN commits to the Asterisk project]

Author: rmudgett
Revision: 411702
Modified property: svn:log

Modified: svn:log at Fri Apr  4 13:38:53 2014
------------------------------------------------------------------------------
--- svn:log (original)
+++ svn:log Fri Apr  4 13:38:53 2014
@@ -1,4 +1,33 @@
-locking: Add temporary sanity checks.
+internal_timing: Remove the option and always make it enabled if a timing module is loaded.
 
-Add some temporary sanity checks to hunt for locking problems with the
-masquerade supertest.
+The masquerade supertest frequently fails because either the local channel
+chain doesn't completely optimize out or the DTMF handshake doesn't
+completely get accross.  Local channel optimization requires frames
+flowing to trigger when optimization can happen.  When optimization
+happens the media frame that triggered the optimization is dropped.
+Sending DTMF requires frames to flow in the other direction for timing
+purposes while sending nothing.  If internal timing is not enabled when
+MOH is playing, Asterisk switches to received timing when an audio frame
+is received.  With optimization dropping media frames and MOH not sending
+frames unless it receives frames, occasionaly there are no more frames
+being passed and the test fails.
+
+* The asterisk command line -I option and the asterisk.conf
+internal_timing option are removed.  Asterisk now always uses internal
+timing when needed if any timing module is loaded.  The issue
+ASTERISK-14861 did this quite awhile ago in v1.4 but effectively is broken
+if other internal timing modules besides DAHDI are used.  The
+ast_read_generator_actions() now only does received timing if it has no
+choice for frame generators like MOH, silence, and playback streaming.
+
+* Cleaned up some code dealing with frame generators in
+ast_deactivate_generator(), generator_write_format_change(),
+ast_activate_generator(), and ast_channel_stop_silence_generator().
+
+ASTERISK-22846 #close
+Reported by: Matt Jordan
+
+Review: https://reviewboard.asterisk.org/r/3414/
+........
+
+Merged revisions 411715 from http://svn.asterisk.org/svn/asterisk/branches/1.8