[FFmpeg-devel] [PATCH 1/4] avfilter/vf_sr.c: refine code to use AVPixFmtDescriptor.log2_chroma_h/w

Guo, Yejun yejun.guo at intel.com
Sat Feb 29 06:48:53 EET 2020



> -----Original Message-----
> From: Guo, Yejun
> Sent: Tuesday, February 25, 2020 2:07 PM
> To: Pedro Arthur <bygrandao at gmail.com>; FFmpeg development discussions
> and patches <ffmpeg-devel at ffmpeg.org>
> Subject: RE: [FFmpeg-devel] [PATCH 1/4] avfilter/vf_sr.c: refine code to use
> AVPixFmtDescriptor.log2_chroma_h/w
> 
> 
> 
> > -----Original Message-----
> > From: Pedro Arthur [mailto:bygrandao at gmail.com]
> > Sent: Monday, February 24, 2020 11:30 PM
> > To: FFmpeg development discussions and patches
> <ffmpeg-devel at ffmpeg.org>
> > Cc: Guo, Yejun <yejun.guo at intel.com>
> > Subject: Re: [FFmpeg-devel] [PATCH 1/4] avfilter/vf_sr.c: refine code to use
> > AVPixFmtDescriptor.log2_chroma_h/w
> >
> > Em seg., 24 de fev. de 2020 às 05:50, Guo, Yejun <yejun.guo at intel.com>
> > escreveu:
> > >
> > > Signed-off-by: Guo, Yejun <yejun.guo at intel.com>
> > > ---
> > >  libavfilter/vf_sr.c | 40 ++++++----------------------------------
> > >  1 file changed, 6 insertions(+), 34 deletions(-)
> > >
> > > diff --git a/libavfilter/vf_sr.c b/libavfilter/vf_sr.c
> > > index 562b030..f000eda 100644
> > > --- a/libavfilter/vf_sr.c
> > > +++ b/libavfilter/vf_sr.c
> > desc->log2_chroma_w);
> > > +
> > >              sr_context->sws_contexts[0] =
> sws_getContext(sws_src_w,
> > sws_src_h, AV_PIX_FMT_GRAY8,
> > >
> > sws_dst_w, sws_dst_h, AV_PIX_FMT_GRAY8,
> > >
> > SWS_BICUBIC, NULL, NULL, NULL);
> > > --
> > > 2.7.4
> > >
> > LGTM
> 
> Thanks.
> 
> Since there is an issue in patch 3 and patch 4, I will send out V2 patch set
> without this one.

I'll push this patch (not this patch set) next Monday if no other comments. thanks.

> 
> >
> > > _______________________________________________
> > > ffmpeg-devel mailing list
> > > ffmpeg-devel at ffmpeg.org
> > > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> > >
> > > To unsubscribe, visit link above, or email
> > > ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list