[FFmpeg-devel] [PATCH] libmpcodecs support

Luca Barbato lu_zero
Fri Jan 14 08:04:01 CET 2011


On 1/14/11 5:37 AM, Michael Niedermayer wrote:
> Hi
>
> Attached patchset makes libavfilter support most libmpcodecs filters
> The libmpcodecs filters are practically unmodified and for keeping it
> maintainable id like to keep them unmodified.

In order to keep things sane, please DO NOT.
The swscale experience is fresh again for me and I'd rather not.

> I will commit this soon but wont enable it yet. That way others can help
> cleanup the wraper (minus code that for maintainability should stay identical
> to the original), fix bugs, rename all global functions so they wont conflict
> with mplayers if they ever include this and generally help.

> Please dont bikeshed this to death.

Mind keeping that in a branch so people interested may get it and people 
scared can live w/out? I'm not sure if that code is less scary than 
swscale BUT your following comment says much.

> Ahh before i can commit this, libmpcodecs and subdirectories must not get
> blocked by the tab&  trailing whitespace checkin scripts.

Are you joking? If you are _really_ serious I'd suggest (or even I'd do 
myself) to clean it up at least to be properly indented in mplayer.

So in short, to cut all the bikeshed and in the as calm as possible tone:

since directly importing code from mplayer collides with:
- coding style
- basic respect for structured code
- people that might be interested in contribute optimizations

I'd rather have it polished before, either in mplayer or here, if you 
are itching to commit it, make a branch and advertise it so before it 
hits the master those 3 points might be addressed already.

lu



More information about the ffmpeg-devel mailing list