[vlc-devel] Is "live.patch" still needed?
jean-paul.saman at planet.nl
Thu Dec 27 15:36:55 CET 2007
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.
> - <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 ?
More information about the vlc-devel