[gst-devel] make the filesrc deal with files whose size is growing

[Christophe Fergeau]

Le jeu 25/09/2003 à 15:08, in7y118 at public.uni-hamburg.de a écrit :
> Stupid question: Does this break length querying, because elements/applications 
> assume length to be fixed once queried?

If some other elements/applications cache the result of a length query
because they expect it to remain constant, they will indeed break. Otoh,
apps or elements caching the length will probably behave the same with
or without this change, they won't take into account the fact that the
file src length changes. 

I don't know if there are parts of gstreamer relying on this length not
changing, that's one of the reasons of this email.

[some background on this patch, a bit off-topic] Originally, I wrote
this patch because I needed it for some work project using very simple
gstreamer pipelines. I thought I'd submit it on this list in case it
would be of interest, it's in no way a patch I took months to write
evaluating the impact of the change on all gstreamer elements and stuff.

Christophe

> Which brings forth another thing: What currently is done via queries should 
> probably be done via something like signals, because you never know when to 
> query again (this holds especially for position queries, too)...
> 
> I guess I should file a bug about this...
> 
> Benjamin
> 
> 
> Quoting David Schleef <ds at schleef.org>:
> 
> > On Wed, Sep 24, 2003 at 09:41:53AM +0200, Christophe Fergeau wrote:
> > > Hi,
> > > 
> > > I needed to play a file with gstreamer while it was being ripped, and I
> > > noticed that the filesrc didn't properly take into account files whose
> > > size is growing while it is processing them. I came up with the attached
> > > patch to handle that. 
> > > The patch is quite ugly imo, so I was wondering if someone had better
> > > ideas to handle that.
> > 
> > Seems ok to me.  Adding a object property is bogus, though, since
> > files can always change size.  update_filelen() should probably
> > be called gst_filesrc_check_file_size().  Also, when seeking,
> > filesrc should unconditionally update the file length.
> > 
> > 
> > 
> > dave...
> > 
> > 
> > 
> > -------------------------------------------------------
> > This sf.net email is sponsored by:ThinkGeek
> > Welcome to geek heaven.
> > http://thinkgeek.com/sf
> > _______________________________________________
> > gstreamer-devel mailing list
> > gstreamer-devel at lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/gstreamer-devel
> > 
> 
> 
> 
> 
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> gstreamer-devel mailing list
> gstreamer-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/gstreamer-devel