[MPlayer-dev-eng] [PATCH] New commands for loading/unloading audio filters at runtime

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sat Apr 3 16:31:18 CEST 2010


On Fri, Apr 02, 2010 at 09:27:21PM +0900, Jehan Pagès wrote:
> I already sent the patch a few days ago but maybe my messages went
> over the radar as I had no answer, then I have just seen the patch doc
> and figures it may be why I had no answer (especially not naming my
> previous emails with [PATCH]). So I try again by following all the
> rules (at least hopefully, tell me if I still do something wrong). :)

It's more that almost nobody except me reviews patches, and I'm
a bit short on time.
I guess it's a somewhat nice feature, but it should be clearly
documented in DOCS/slave.txt that these are experimental.
While those functions are there, only the af_add one is somewhat
tested (but still had a bugs for years that was only recently fixed),
anyone who uses them extensively will run into major bugs.
I'd even suggest to recommend that any application using them
should provide a way to make it run without using them, too.



More information about the MPlayer-dev-eng mailing list