Alexander Chemeris wrote:
> Thanks for good bugreport. I commited fix to sipXtapi branch, could you
> test it or you use media-update branch?
>
> It is better to create issues in Jira to easier keep track of bugs and where
> they fixed and where not.
>

I currently use sipxtapi branch.

I still cant create issues in Jira. I see only "home", "browse projects" and 
"find issues". I also tried to register again 2 weeks ago and the other account 
is the same.

Your code in MpdSipxPcma.cpp is wrong btw

in MpdSipxPcma.h you say
"bool mIsFirstFrame; ///< True, if no frames decoded."

It's true if we havent decoded any frames yet, but you set it to false in 
constructor and elsewhere (there are 2 lines mIsFirstFrame = false; there). So 
the condition if (mIsFirstFrame) will never be met. You have to initialize it 
to true.

I also saw the line JB_initCodepoint(pJBState, "PCMA", 8000, getPayloadType()); 
in the patch. Are there 2 jitter buffers in the code? In my proposal using 
sorted linked list there would be need for only 1 jitter buffer for each codec. 
Also speex and gsm would use it. In my linked list implementation the list is 
in MprDejitter.

I also detected another bug somewhere in the code, when I use nortel VPN 
client, I dont hear any sound even with my patch but because no rtp data from 
listening socket is arriving. I guess the socket is listening on wrong ip 
somehow. I don't have time to investigate this, but an issue should be created 
for it. I want to get multiple lines, conferences, call recording and basic 
call stuff like that working in my softphone first.

I was happy that I found http://sipx-wiki.calivia.com/index.php/SipXmediaLib . 
Without it I would be lost to find that bug. When I have better knowledge of 
flowgraphs I might attempt to modify it a little bit so it works with multiple 
instances of sipxtapi. Too many modifications without proper knowledge lead to 
too many bugs ;-)

Jaro
_______________________________________________
sipxtapi-dev mailing list
[email protected]
List Archive: http://list.sipfoundry.org/archive/sipxtapi-dev/

Reply via email to