[asterisk-dev] [Code Review] awesome review

[Kevin P. Fleming]

On 09/20/2010 07:06 AM, Vadim Lebedev wrote:
> On 09/20/2010 11:03 AM, Klaus Darilion wrote:
>>
>> Am 17.09.2010 21:05, schrieb David Vossel:
>>   
>>> ----- Original Message -----
>>>     
>>>> From: "Mark Murawski"<markm at intellasoft.net>
>>>> To: asterisk-dev at lists.digium.com
>>>> Sent: Friday, September 17, 2010 1:12:03 PM
>>>> Subject: Re: [asterisk-dev] [Code Review] awesome review
>>>> I would suggest a different name. What about AudioFrameHook ?
>>>>       
>>> This hook is not specific to only audio frames. Plus that name is not very awesome.
>>>     
>> I would also suggest a more meaningful name - even if it is awesome. 
>> Although I do not know what it exactly does, what about:
>>
>> GenericMediaHook?
>> GenericFrameHook?
>> FrameHook
>> AstFrameHook
>> MediaHook?
>> AwesomeFrameHook?
>> AwesomeAstFrameHook?
>>   
> 
> Klaus,
> 
> You forgot the obvious:
> MediaFrameHook

No, the whole point of this is API is that it is *not* restricted to
media frames. I've already suggested 'ChannelShim' to David, which I
think is the best way to describe what this API does.

-- 
Kevin P. Fleming
Digium, Inc. | Director of Software Technologies
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
skype: kpfleming | jabber: kfleming at digium.com
Check us out at www.digium.com & www.asterisk.org