[FFmpeg-devel] [PATCH] swscale: remove unused variable from context.

Ramiro Polla ramiro.polla
Tue Jul 21 22:17:47 CEST 2009


On Mon, Jul 20, 2009 at 3:03 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
> On Mon, Jul 20, 2009 at 02:57:14PM -0300, Ramiro Polla wrote:
>> On Mon, Jul 20, 2009 at 12:21 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
>> > On Sun, Jul 19, 2009 at 01:34:34AM -0300, Ramiro Polla wrote:
>> >> I don't think these variables were ever used. I can't make sense of
>> >
>> > from what i remember wihout checking (might be wrong) they represent the
>> > chroma subsample amount of the internal buffer (that is betweem horizontal
>> > and vertical scaling
>>
>> Ok, but they're just a copy of chrSrc{H,V}SubSample.
>
> yes and this is maybe wrong ...
>
>
>> ?None of them are
>> altered afterwards and chrInt{H,V}SubSample are never used.
>
> true but maybe they could be used to simplify some code, until then
> i dont really mind f they are removed ...

Removed for now.



More information about the ffmpeg-devel mailing list