[vlc-devel] commit: live555: higher caching, corrects timestamp with caching ( Rafaël Carré )

[Jean-Paul Saman]

Derk-Jan Hartman wrote:
> On 22 sep 2008, at 10:45, git version control wrote:
>> vlc | branch: 0.8.6-bugfix | Rafaël Carré <rcarre at m2x.nl> | Mon Sep  
>> 22 10:48:42 2008 +0200| [dd6a118abe0c964efa7cd633e734744cc27ec29e] |  
>> committer: Rafaël Carré
>>
>> live555: higher caching, corrects timestamp with caching
>>
>>> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=dd6a118abe0c964efa7cd633e734744cc27ec29e
>> ---
>>
>> modules/demux/live555.cpp |   15 ++++++++++++---
>> 1 files changed, 12 insertions(+), 3 deletions(-)
>>
>> diff --git a/modules/demux/live555.cpp b/modules/demux/live555.cpp
>> index 90040d3..89ff69b 100644
>> --- a/modules/demux/live555.cpp
>> +++ b/modules/demux/live555.cpp
>> @@ -110,7 +110,7 @@ vlc_module_begin();
>>                   N_("Port to use for tunneling the RTSP/RTP over  
>> HTTP."),
>>                   VLC_TRUE );
>> #endif
>> -        add_integer("rtsp-caching", 4 * DEFAULT_PTS_DELAY / 1000,  
>> NULL,
>> +        add_integer("rtsp-caching", 12 * DEFAULT_PTS_DELAY / 1000,  
>> NULL,
>>                     CACHING_TEXT, CACHING_LONGTEXT, VLC_TRUE );
>>         add_bool(   "rtsp-kasenna", VLC_FALSE, NULL, KASENNA_TEXT,
>>                     KASENNA_LONGTEXT, VLC_TRUE );
> 
> Huh ?
> 
> That does not seem right to me. Is there a bug in VLC or in live ?
> And why does this "fix" it ? Also is this something we want in a 0.8.6- 
> bugfix tree ?

This commit does look strange too me too. However I am sure Rafaël did a 
lot of testing on this. It sounds to me like there is an issue in live.

Rafaël could you send a test link for this, so thedj can try and observe 
this issue himself too?

Gtz
Jean-Paul Saman.