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

Michael Niedermayer michaelni
Mon Jul 20 20:03:11 CEST 2009

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 ...


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090720/98dac0f3/attachment.pgp>

More information about the ffmpeg-devel mailing list