[MPlayer-dev-eng] [PATCH] for starting .rm streaming over rtsp:// at specifiedstarting point(.ram syntax)

[Roberto Togni]

On 2003.08.03 06:46, Bernhard Kaindl wrote:
> Hi,
>    I needed to start rtsp://-streamed realvideo&audio with some start
> offset in the video clip like it's often used in .ram files like this
> 
> rtsp://helixserver.example.com/video1.rm?start="0:51:02"
> 
> to start realplay playback at offset 51 min 02 sec into the
> stream(e.g.)
> 
...
> 
> So I think the patch should not have a problem to be included.
> 
Looks good, but i think we should not remove the parameters from the 
url. Are you sure that the server never needs the parameters?

Can you send an url where i can test it?

> I also did a small change to *rtsp_get() which was aborting mplayer
> with
> the message "librtsp: buffer overflow in rtsp_get\n" when playing an
> rtsp:// stream over UDP in combination with a server which does not
> get even a single answer to mplayer for a few seconds. The cause was
> that the for loop loops pretty quick and ran quickly over it's
> treshold.
> 
> It would likely be best to run the recv() using an timeout, with this
> change it loops untis a newline is received from the server.
If i understand it correctly, you're saying that it will never exit the 
loop if it doesn't get data from the server. This is not good IMHO.
Can some network guru comment on this?

> 
> Bernhard
Ciao,
  Roberto