[asterisk-dev] [asterisk-commits] mmichelson: branch mmichelson/digit_manipulation r196953 - /team/mmichelson/d...

[Mark Michelson]

Kevin P. Fleming wrote:
> SVN commits to the Asterisk project wrote:
> 
>> Modified: team/mmichelson/digit_manipulation/main/channel.c
>> URL: http://svn.asterisk.org/svn-view/asterisk/team/mmichelson/digit_manipulation/main/channel.c?view=diff&rev=196953&r1=196952&r2=196953
>> ==============================================================================
>> --- team/mmichelson/digit_manipulation/main/channel.c (original)
>> +++ team/mmichelson/digit_manipulation/main/channel.c Tue May 26 18:14:07 2009
>> @@ -5013,8 +5013,13 @@
>>  			case AST_CONTROL_T38:
>>  			case AST_CONTROL_SRCUPDATE:
>>  			case AST_CONTROL_CONNECTED_LINE:
>> -				if (ast_channel_connected_line_macro(who, other, f, other == c0, 1))
>> -					/* Yes, this is a purposeful fallthrough. ewwww */
>> +				if (ast_channel_connected_line_macro(who, other, f, other == c0, 1)) {
>> +					ast_indicate_data(other, f->subclass, f->data.ptr, f->datalen);
>> +				}
>> +				if (jb_in_use) {
>> +					ast_jb_empty_and_reset(c0, c1);
>> +				}
>> +				break;
> 
> This still seems broken, as it is calling
> ast_channel_connected_line_macro() for AST_CONTROL_T38 and
> AST_CONTROL_SRCUPDATE frames.
> 
> 

Yeppers.

Mark Michelson