[asterisk-commits] seanbright: branch 10 r355623 - in /branches/10: ./ main/audiohook.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Feb 16 14:02:02 CST 2012


Author: seanbright
Date: Thu Feb 16 14:01:59 2012
New Revision: 355623

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=355623
Log:
Revert a change to audio_audiohook_write_list that had no affect.

When I made this change initially, I was under the false impression that the
audiohooks structure remained on the channel after all of the hooks had been
detached.  This is not the case, ast ast_read takes care of removing the
audiohooks structure if the lists are empty.
........

Merged revisions 355622 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/10/   (props changed)
    branches/10/main/audiohook.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/main/audiohook.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/main/audiohook.c?view=diff&rev=355623&r1=355622&r2=355623
==============================================================================
--- branches/10/main/audiohook.c (original)
+++ branches/10/main/audiohook.c Thu Feb 16 14:01:59 2012
@@ -783,10 +783,6 @@
 	int middle_frame_manipulated = 0;
 	int removed = 0;
 
-	/* Don't translate our frame if we aren't going to bother to use it */
-	if (ast_audiohook_write_list_empty(audiohook_list))
-		return end_frame;
-
 	/* ---Part_1. translate start_frame to SLINEAR if necessary. */
 	if (!(middle_frame = audiohook_list_translate_to_slin(audiohook_list, direction, start_frame))) {
 		return frame;




More information about the asterisk-commits mailing list