[FFmpeg-devel] [PATCH] Add NULL mixer to AVSequencer.

Ronald S. Bultje rsbultje
Mon Aug 16 15:13:44 CEST 2010


Hi,

On Mon, Aug 16, 2010 at 2:22 AM, Sebastian Vater
<cdgs.basty at googlemail.com> wrote:
> We should utilize multiple mixers any way. Since we want to target
> multiple audiences, right?

Do we want multiple mixers? I'm trying to make the point that we
don't. If the quality of a S32 mixer is poor, than your mixer isn't
good enough and we need to improve it before committing it. I've never
heard of anyone wanting to sacrifice 99% of the sound quality for a 2x
speedup, so to say.

Try to write your "null mixer" as a if (!buf) in the high-quality
mixer, that should suffice for both cases, saves >1000 lines of code
and means we don't need a synth mixer registration API and versioning
symbols anymore. Also, only do this if this is strictly required for
TCM1. If there's other ways to accomplish the same thing in TCM1,
don't focus on the null mixer, since then we clearly don't need it.

Ronald



More information about the ffmpeg-devel mailing list