[FFmpeg-devel] [PATCH] libmpcodecs support

Luca Barbato lu_zero
Sat Jan 15 16:47:56 CET 2011


On 1/15/11 12:25 PM, Michael Niedermayer wrote:
> On Sat, Jan 15, 2011 at 03:17:24AM +0100, Luca Barbato wrote:
>> On 1/14/11 3:19 PM, Michael Niedermayer wrote:
> [...]
>>> i can just merge the changes in, but IMHO if it takes you more than a few days
>>> to run indent or sed over the code and get it approved then waiting longer has
>>> zero sense. Because indent takes seconds, and getting it approved either will
>>> happen or will not happen. And its twice as much code than there is in swcale so
>>> doing anything by hand is (from experience of swscale) something taking years
>>> and i think we all agree that blocking 80% of functionality of libavfilter
>>> for years is not worth the style changes to code that is semantically not even
>>> part of ffmpeg.
>>
>> I'd rather have clean and maintainable code in ffmpeg. I could accept
>> this kind of devils deal only if it gets first a full regression test so
>> trying to clean it up later would be half of the work.
>
> dont forget the full regression test for swscale as condition to your
> cleanup there

I'm assuming the one currently present covers everything, point me to 
what's missing if I'm wrong, I didn't check while running it so far ^^;

lu



More information about the ffmpeg-devel mailing list