[FFmpeg-devel] [PATCH] vf_psnr: fix rgb channel order in metadata names.

[Ronald S. Bultje]

Hi,

On Sat, Jul 11, 2015 at 1:03 AM, Paul B Mahol <onemda at gmail.com> wrote:

> Dana 11. 7. 2015. 04:50 osoba "Ronald S. Bultje" <rsbultje at gmail.com>
> napisala je:
> >
> > ---
> >  libavfilter/vf_psnr.c | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> >
> > diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c
> > index 0c2c950..d9a6030 100644
> > --- a/libavfilter/vf_psnr.c
> > +++ b/libavfilter/vf_psnr.c
> > @@ -289,9 +289,9 @@ static int config_input_ref(AVFilterLink *inlink)
> >      }
> >
> >      s->is_rgb = ff_fill_rgba_map(s->rgba_map, inlink->format) >= 0;
> > -    s->comps[0] = s->is_rgb ? 'r' : 'y' ;
> > -    s->comps[1] = s->is_rgb ? 'g' : 'u' ;
> > -    s->comps[2] = s->is_rgb ? 'b' : 'v' ;
> > +    s->comps[0] = s->is_rgb ? 'g' : 'y' ;
> > +    s->comps[1] = s->is_rgb ? 'b' : 'u' ;
> > +    s->comps[2] = s->is_rgb ? 'r' : 'v' ;
> >      s->comps[3] = 'a';
> >
> >      s->planeheight[1] = s->planeheight[2] = FF_CEIL_RSHIFT(inlink->h,
> desc->log2_chroma_h);
> > --
> > 2.1.2
> >
>
> See rgba_map above, this breaks code that use it.


Oh, I see, I didn't notice that, I guess patch withdrawn then.

Ronald