>To tell the full story, there is another little issue with multiplexors,
>since {audio,video}_out_file should be considered as BASENAMES, not full
>paths, so avi multiplexor can attach it's own extension, so raw multiplexor,
>ogm one and so on. I'll do this change into near future.
Please, no! I spent so much time getting rid of the extension-adding
code because it kept breaking my scripts. :( If the user wants .avi or
.mpg or whatever on his filenames, let the user put it there himself. You
never know, maybe some obscure app will only work with MPEG streams that
have the extension ".bob"...
>To get back on open issues, in facts current situation it's counter intuitive,
>maybe clumsy, surely suboptimal.
>I don't already add another method since I'm quite worried about method
>explosion, I want to avoid it.
I agree here, and I was trying to think of a way to take care of
things without adding a new function, but you'd end up having to have an
if-block in every (de)mux function to see if the files were opened, so it
looks like it'll be cleaner to add a new function instead.
>- add int (*open)(TCModuleInstance *self,
> const char *video_file, const char *audio_file);
>/* or maybe s/video/primary/ s/audio/auxiliary/ ? */
>
>that open each given file if argument != NULL and returns -1 if error
>(tc_log*()'in it) and N if succesfull, where N it's equal to open files)
Hmm... I'd think it would be simpler to just pass a single filename,
because usually you'll be using a different demuxer/muxer for the video and
audio files if you keep them separate (e.g. mux_avi for video, mux_wav for
audio). The core would create two (de)muxer instances, one for each
separate file, and pass just the video frame or just the audio frame to the
(de)mux routine as appropriate.
>A word about stop() method.
>It will desiderable that we decouples file flushing/closing operation from
>general cleanup routines?
Yes, I agree with this--if nothing else, it makes sense to have a
close() as long as we have an open().
--Andrew Church
[EMAIL PROTECTED]
http://achurch.org/
