[FFmpeg-soc] [soc] libavsequencer [PATCH 02/08] Sub-song public API header file.

[Ronald S. Bultje]

Hi,

On Sun, Aug 15, 2010 at 3:55 PM, Sebastian Vater
<cdgs.basty at googlemail.com> wrote:
> But shouldn't be the rule here, if there is one exception that we have
> to address that issue?

I think we're too much exception-driven here. Yuo might want to focus
on a BSS that is sufficient for TCM files if that's what your SoC
end-goal is. This will not be ABI/API-stable, and we'll extend it with
new features as you add support for more mod files.

However, right now this is greatly over-engineered and
over-complicated because you're trying to squash in support for all
obscure features of all mod files. Some of these might never land in
FFmpeg. Review is impossible because these features aren't actually
used yet.

If TCM is what you're working on, then please focus on what TCM does,
evolve-as-needed. No more. That way it's easier for all of us. And no
tracker features please, focus on playback only. Tracker features will
be added as trackers based on FFmpeg start to be written and we know
what's required for them to work correctly. Again, impossible to
review right now, because unused.

Ronald