[FFmpeg-devel] [PATCH 2/2] lavfi/lut: simplify nested lut plane stepping.

Clément Bœsch ubitux at gmail.com
Wed Apr 17 19:29:03 CEST 2013


On Wed, Apr 17, 2013 at 10:22:16AM +0000, Paul B Mahol wrote:
> On 4/17/13, Clement Boesch <ubitux at gmail.com> wrote:
> > ---
> >  libavfilter/vf_lut.c | 14 +++++---------
> >  1 file changed, 5 insertions(+), 9 deletions(-)
> >
> > diff --git a/libavfilter/vf_lut.c b/libavfilter/vf_lut.c
> > index be497ac..21309a5 100644
> > --- a/libavfilter/vf_lut.c
> > +++ b/libavfilter/vf_lut.c
> > @@ -286,15 +286,11 @@ static int filter_frame(AVFilterLink *inlink, AVFrame
> > *in)
> >              inrow  = inrow0;
> >              outrow = outrow0;
> >              for (j = 0; j < w; j++) {
> > -                outrow[0] = tab[0][inrow[0]];
> > -                if (lut->step>1) {
> > -                    outrow[1] = tab[1][inrow[1]];
> > -                    if (lut->step>2) {
> > -                        outrow[2] = tab[2][inrow[2]];
> > -                        if (lut->step>3) {
> > -                            outrow[3] = tab[3][inrow[3]];
> > -                        }
> > -                    }
> > +                switch (lut->step) {
> > +                case 3:  outrow[3] = tab[3][inrow[3]]; // Fall-through
> > +                case 2:  outrow[2] = tab[2][inrow[2]]; // Fall-through
> > +                case 1:  outrow[1] = tab[1][inrow[1]]; // Fall-through
> > +                default: outrow[0] = tab[0][inrow[0]];
> >                  }
> >                  outrow += lut->step;
> >                  inrow  += lut->step;
> > --
> > 1.8.2.1
> >
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> 
> lgtm

Applied.

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130417/1821738b/attachment.asc>


More information about the ffmpeg-devel mailing list