[FFmpeg-devel] [PATCH][RFC] Lagarith Decoder.

Nathan Caldwell saintdev
Sat Aug 22 22:36:58 CEST 2009


2009/8/22 M?ns Rullg?rd <mans at mansr.com>:
> Nathan Caldwell <saintdev at gmail.com> writes:
>
>> 2009/8/21 M?ns Rullg?rd <mans at mansr.com>:
>>> Nathan Caldwell <saintdev at gmail.com> writes:
>>>> + * @file libavcodec/lagrange.c
>>>
>>> That filename makes it look like it has something to do with Lagrange,
>>> which appears not to be the case, at least not directly.
>>
>> Would lagarithrac.c perhaps be better?
>
> Why is this in a separate file anyway? ?Is it useful to other codecs?

Good question, I asked that myself. Originally it wasn't separate, I
only did that because Michael suggested it. The answer I got from
Diego was:

> 1) Smaller patches are quicker to get reviewed and applied.
> 2) We try to support a modular build as much as possible.

As far as I know it's not useful to any other codec.



-- 
-Nathan Caldwell



More information about the ffmpeg-devel mailing list