[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