OK, I just found the audio glitches problem... I disabled LINEAR_COMPLEXITY_BRIDGE in MprBridge.h --> No more audio glitches !
I fear there must be another problem in the new bridge implementation !!! stipus ----- Original Message ----- From: "stipus" <[EMAIL PROTECTED]> To: "Keith Kyzivat" <[EMAIL PROTECTED]> Cc: <[email protected]> Sent: Tuesday, July 24, 2007 6:31 PM Subject: Re: [sipxtapi-dev] Bug in mpCallFlowGraph with DISABLE_LOCAL_AUDIOin 9768 > My code *was* using rev 9768 (so no callRecSplitters). > In the beginning of the afternoon, I decided to SVN update to latest > revision (rev 9887 when I updated). > > --> As my application is a server ( DISABLE_LOCAL_AUDIO is defined), I got > one bug each time calls are hangup. This is because a #ifdef is missing when > removing links (line 550 of mpCallFlowGraph - see below). > > --> My server is using SipxCallPlayBufferStart() to play audio messages. > Before I updated (I mean with rev 9768) I had no audio glitches. > With newest 9887, I get many audio glitches. I'm still trying to understand > what's happening... > > stipus > > > ----- Original Message ----- > From: "Keith Kyzivat" <[EMAIL PROTECTED]> > To: "stipus" <[EMAIL PROTECTED]> > Cc: <[email protected]> > Sent: Tuesday, July 24, 2007 6:21 PM > Subject: Re: [sipxtapi-dev] Bug in mpCallFlowGraph with DISABLE_LOCAL_AUDIO > in 9768 > > > > I'm confused a bit here.. > > mpSpeakerCallrecSplitter was not introduced until 9772.. > > I'm guessing you're on a later revision than 9768. > > > > Can you please update your tree to the latest code, and create a patch > using > > 'svn diff '? > > > > svn update > > svn diff <path to your sipXtapi working copy> > > will do that. > > > > > > If you think that the new call recording feature is causing problems, you > > can try the latest code without it by using svn merge to subtract it out: > > > > svn merge -r 9772:9771 <path to your sipXtapi working copy> > > > > On 7/24/07, stipus <[EMAIL PROTECTED]> wrote: > > > > > > mpCallFlowGraph line 281: > > > > > > #ifndef DISABLE_LOCAL_AUDIO > > > > > > res = addLink(*mpSpeakerCallrecSplitter, 0, *mpToSpkr, 0); > > > > > > assert(res == OS_SUCCESS); > > > > > > #endif > > > > > > > > > > > > Line 550: > > > > > > res = removeLink(*mpSpeakerCallrecSplitter, 0); assert(res == > OS_SUCCESS); > > > > > > > > > > > > > > > I think there are missing #ifdef ....... should be a lot better like > this: > > > > > > #ifndef DISABLE_LOCAL_AUDIO > > > > > > res = removeLink(*mpSpeakerCallrecSplitter, 0); assert(res == > OS_SUCCESS); > > > > > > #endif > > > > > > > > > > > > BTW, I just updated my code from 9768 to 9887, and I get many audio > > > glitches. Rolling back to 9768 solves the problem... Maybe all these new > > > recorder Splitters and Mixers in the flowgraph create this problem... ? > > > > > > > > > > > > stipus > > > > > > > > > > > > _______________________________________________ > > > sipxtapi-dev mailing list > > > [email protected] > > > List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/ > > > > > > > > > > > -- > > Keith Kyzivat > > > > SIPez LLC. > > SIP VoIP, IM and Presence Consulting > > http://www.SIPez.com > > tel: +1 (617) 273-4000 > > > > _______________________________________________ > 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/
