On 2/7/07, Jaroslav Libák <[EMAIL PROTECTED]> wrote:
> 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.
I'll try ping admins once more.

> 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.
Ooops... It was really time to sleep. :) Fixed in svn, thanks.

> 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.
This system was designed to enable codecs-specific JBs. AFAIK it may be the
case for commercial codecs - they may include own JB+PLC. I'll consult with
man, who should know do we need this or not. It is one of the reasons, why
I want see all discussions on the list - many people may participate. :) Other
pros is that this question will be unlickely asked again - people use search
through archives.

> 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.
*nod* It is known. For now - try using 'szBindToAddr' parameter in
sipXinitialize()
call. Look into PlaceCall for example usage.

> I was happy that I found http://sipx-wiki.calivia.com/index.php/SipXmediaLib .
> Without it I would be lost to find that bug.
Very glad to hear it helped :) Though, text on this page is a bit outdated and
should be updated and improved. I had litle time for this, but I've already
put updated picture a week ago - it shows flowgraph internals correctly.

> 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 ;-)
Sure. But multiplу instances may will likely face more problems outside
flowgraphs. Libraries still have statically initialized variables and other
nasty things. I cleaned up some of them, but not all. Hope this will not
put you away.

--
Regards,
Alexander Chemeris.

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

Reply via email to