[Mplayer-cvslog] CVS: main/libmpdemux stream.c,1.52,1.53

[Alban Bedel]

Hi Ross Finlayson CVS,

on Wed, 2 Apr 2003 03:46:51 +0200 you wrote:

> Update of /cvsroot/mplayer/main/libmpdemux
> In directory mail:/var/tmp.root/cvs-serv16575
> 
> Modified Files:
> 	stream.c 
> Log Message:
> Added a test for "s->fd == NULL" to the start of "stream_fill_buffer()".
> ("fd" is NULL if the session is defined by a SDP file, for example.)
> It turns out that this test is necessary only for Solaris, not Linux,
> because in Linux, "s->eof" got set after the prior (failing) call to
> "lseek()" in "stream_seek_long()".  In Solaris, however, "lseek(NULL,
> ...)" returns 0 (go figure!) rather than -1, so "s->eof" doesn't get set
> in"stream_seek_long()".
Imho this should then be fixed in stream_seek_long (as it's seems to be the
problem source) to have a consistante behaviour.
	Albeu