Hi,

Sure, it works. No more work is needed except for documentation.

Regards,
Kenichi Aramaki


2006/9/10, Alexander Chemeris <[EMAIL PROTECTED]>:
> Hello,
>
> On 9/9/06, Kenichi Aramaki <[EMAIL PROTECTED]> wrote:
> > Though MpCodec_setGain() looks like to expect a value from 1 to 10
> > which is described in CpMediaInterfaceFactoryImpl.h,
> > sipxAudioSetGain() passes 100 scale value to MpCodec_setGain().
> > Since 10 scale is rough, how about changing to 100 scale?
> > (why they starts with 1, not 0?)
> >
> > These are several magic numbers (like 1.0f, 9.0f) in MpCodec_setGain(),
> > they should be symbols. If we use float conversion (As dwMaximum is
> > typically 0xFFFF, I think it is not essential) the correspond code can be
> > as follows;
> I've looked through the code and it looks like you've described. Does
> setGain
> work with changes you provided, or there are more work to be done? If it
> works,
> I'll commit it tomorrow.
>
> Regards,
> Alexander Chemeris.
> _______________________________________________
> sipxtapi-dev mailing list
> [email protected]
> List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/
>
_______________________________________________
sipxtapi-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/

Reply via email to