[FFmpeg-devel] [PATCH 11/12] AVSequencer player handler of BSS for libavsequencer

[Sebastian Vater]

Ronald S. Bultje a ?crit :
> Hi,
>
> On Sat, Aug 7, 2010 at 4:27 PM, Sebastian Vater
> <cdgs.basty at googlemail.com> wrote:
>   
>> AVSequencer compiles as in the normal build process, you can turn off
>> the null mixer and/or the low quality mixer using --disable-mixer=NAME
>> --disable-mixers as usual.
>>
>> Here are the player.h of the avsequencer to review.
>>     
>
> I'm confused what the function of "player.h" in BSS would be? BSS is
> our codename for the small chunk of header files ("interface") that
> "describes" a "mod" file so that it can be decoded/demuxer,
> re-encoded/remuxed or synthesized=played_as_PCM. BSS itself is not the
> decoder, muxer or synthesizer/player. So what is the function of
> player.h in all this?
>   

Well, the point is that avsequencer.h doesn't include with the player.h,
the rest is really independant of this.

The includes are required, just the implementation is not. This has to
be be done, to get a whole consistent structure about the avsequencer part.

I surely could, add only the basic stuff to avsequencer.h and add the
player stuff later, but I won't make much sense, since the player API
has to be used in a final state of avsequencer to calculate the
conversation path between src_module and dst_module if converting, see
my very early mails on this.

As written privately, I have currently approx. 90% of the decoder ready
and the demuxer should be on the same line, i.e. it will be only a few
days, when FFplay will be able to playback module files.

All we have to do now is to talk about of the remaining issues. For all
those, who don't have the latest code, please do a git pull (or git pull
--rebase) and get the latest code.

However, the latest code has some caveats. It doesn't compile anymore,
unless you do in your build directory:
../configure --disable-decoder=iff_tcm1 --disable-decoder=seq_tcm1

Please note that the code in seq_tcm.c DOES compile when running under
lavf instead of lavc (after replacing AVCodecContext *avctx with
AVFormatContext *s for the static open_* function calls and prefixing
the metadata get function for these).

-- 

Best regards,
                   :-) Basty/CDGS (-: