[Asterisk-Dev] [patch] SIP jitterbuffer patch?

[Roy Sigurd Karlsbakk]

new patch attached. this should apply cleanly to current cvs and even  
compile without errors

two of the 'fixes' are rather ugly and should be fixed, but we'll see  
how this works for now
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sipjb-2005-06-27.patch
Type: application/octet-stream
Size: 28746 bytes
Desc: not available
Url : http://lists.digium.com/pipermail/asterisk-dev/attachments/20050627/49eddb74/sipjb-2005-06-27.obj
-------------- next part --------------


Den 27. jun. 2005 kl. 12.41 skrev Roy Sigurd Karlsbakk:

> The patch didn't apply 100%, but it was ok to do so manually
>
> however, with current cvs + spandsp, i get the following
>
> rtp.c: In function `get_samples':
> rtp.c:429: warning: implicit declaration of function `g723_samples'
> rtp.c: In function `calc_rxstamp_jb':
> rtp.c:479: warning: unsigned int format, pointer arg (arg 3)
> rtp.c: In function `put_in_jb':
> rtp.c:578: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:584: warning: integer overflow in expression
> rtp.c:586: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:647: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:657: warning: unsigned int format, pointer arg (arg 3)
> rtp.c: In function `get_from_jb':
> rtp.c:715: error: too few arguments to function `jb_get'
> rtp.c:723: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:727: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:730: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:734: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:737: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:804: warning: unsigned int format, pointer arg (arg 3)
> rtp.c:818: warning: unsigned int format, pointer arg (arg 3)
> rtp.c: In function `ast_rtp_jb_enable':
> rtp.c:863: error: structure has no member named `max_jitterbuf'
> rtp.c:864: warning: implicit declaration of function `jb_setinfo'
> rtp.c: At top level:
> rtp.c:96: warning: `rtp_framelog_counter' defined but not used
> make[1]: *** [rtp.o] Error 1
> make[1]: Leaving directory `/usr/src/asterisk+spandsp+sipjb'
> make: *** [dont-optimize] Error 2
> pstngw1:/usr/src/asterisk #
>
>
> Den 27. jun. 2005 kl. 12.01 skrev Slav Klenov:
>
>
>> Actually this patch is made against asterisk cvs-head from  
>> 13.06.2005. I'm not sure it will apply on a cvs-head from today.  
>> Please checkout a 13.06 cvs-head version.
>>
>> Slav Klenov wrote:
>>
>>
>>
>>> I think its there
>>>
>>> try
>>>
>>> http://www.astertest.com/downloads/sip%20jitter%20buffer/latest/ 
>>> jitterbuf-rtp-sip.patch1.patch
>>>
>>>
>>> Roy Sigurd Karlsbakk wrote:
>>>
>>>
>>>
>>>> hi
>>>>
>>>> some weeks ago, i was told there was a new sip jitterbuffer  
>>>> patch to  asterisk cvs head. problem is, i can't find this. it's  
>>>> not at the  usual place, http://www.astertest.com/downloads/sip% 
>>>> 20jitter%20buffer/ latest/
>>>>
>>>> can someone help me out here? i need this to test my new rig
>>>>
>>>> roy
>>>> _______________________________________________
>>>> Asterisk-Dev mailing list
>>>> Asterisk-Dev at lists.digium.com
>>>> http://lists.digium.com/mailman/listinfo/asterisk-dev
>>>> To UNSUBSCRIBE or update options visit:
>>>>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>>>>
>>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> Asterisk-Dev mailing list
>>> Asterisk-Dev at lists.digium.com
>>> http://lists.digium.com/mailman/listinfo/asterisk-dev
>>> To UNSUBSCRIBE or update options visit:
>>>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>>>
>>>
>>
>>
>> _______________________________________________
>> Asterisk-Dev mailing list
>> Asterisk-Dev at lists.digium.com
>> http://lists.digium.com/mailman/listinfo/asterisk-dev
>> To UNSUBSCRIBE or update options visit:
>>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>>
>>
>>
>
> _______________________________________________
> Asterisk-Dev mailing list
> Asterisk-Dev at lists.digium.com
> http://lists.digium.com/mailman/listinfo/asterisk-dev
> To UNSUBSCRIBE or update options visit:
>   http://lists.digium.com/mailman/listinfo/asterisk-dev
>
>