[vlc-devel] Is "live.patch" still needed?

[Jean-Paul Saman]

Ross,

Thank you for applying these changes in your live555 code.

Ross Finlayson wrote:
> I'm hoping that the latest version (2007.12.27) of the "LIVE555 
> Streaming Media" code can make 
> "trunk/extras/contrib/src/Patches/live.patch" unnecessary, thereby 
> simplifiying the VLC build process slightly.
> 
> Specifically:
> - <strstream> is included when available (instead of <strstream.h>) 
> for several releases now.
> - The existing "Locale" class already does the work of 
> "radix_safe_sprintf()" and "radix_safe_sscanf()".  All that was 
> needed was to to change the locale string parameter from "POSIX" to 
> "C".  This is now done in the latest version.
> - The latest version now adds a timeout to the "readSocket()" call in 
> "RTSPClient", as is done in the patch.

I found out that the timeout of 5 seconds is too short in some 
situations, resulting in unneeded retries. A larger timeout works 
better. In VLC it was set to 30 sec by me. (Maybe 30 secs is too large ,..)

What is the status on the live-starttime.patch ?

Gtz,
Jean-Paul Saman.