[MPlayer-dev-eng] Re: [PATCH] Saved AVI index files with -saveidx and -idxfile

[Alban Bedel]

Hi Jason Tackaberry,

on Fri, 10 Oct 2003 09:29:55 -0400 you wrote:

> Hi,
> 
> On Thu, 2003-10-09 at 14:55, Attila Kinali wrote:
> > Jason Tackaberry <tack at auc.ca> wrote:
> > > the index files) and it does things it shouldn't be doing in
> > > libmpdemux(like calling exit()).  I don't expect it to be merged
> > > into cvs without
> [...]
> > Fix those thing, add some sanity checks (remember, mplayer should
> > never segfault), and add docu, than it can be applied.
> 
> I can add sanity checks, documentation, and clean up the code, but I'm
> going to need help from the mplayer developers on removing the exit()
> call.  What is the proper way to cause mplayer to exit within
> libmpdemux?
You shouldn't exit() from libmpdemux. You should return an error code
or the like when failling. Also if the index loading/writing fail it
should only print error message and continue.

BTW this patch give an idea. Having mencoder write this separate
index file regulary and mplayer re-read it everytime it change.
With that it should be possible to seek in a "beeing recorded
avi" :)
Ok, it's hackish ;) But that's a lot easier that writing proper
muxer, and optionaly demuxer, for a format that would support
seeking without index.
	Albeu
-- 

Everything is controlled by a small evil group
to which, unfortunately, no one we know belongs.