Clean and specific patch is always good, agree with you on removal here. On Tue, Mar 20, 2018 at 9:18 AM, Li, Zhong <zhong...@intel.com> wrote:
> > From: libav-devel [mailto:libav-devel-boun...@libav.org] On Behalf Of > > Maxym Dmytrychenko > > Sent: Monday, March 19, 2018 5:56 PM > > To: libav development <libav-devel@libav.org> > > Cc: Liu, ChaoX A <chaox.a....@intel.com> > > Subject: Re: [libav-devel] [PATCH 2/2] lavu/hwcontext_qsv: Add support > for > > pix_fmt RGB32. > > > > indentation can be adjusted , > > It is to support RGB format overlay (e.g. lena-rgba.png). > > > AV_PIX_FMT_YUYV422 - we add it but dont use? > > If the overlay input movie format is 4:2:2, it will be used. But I prefer > to remove it in this patch to make it clearer. > How do you think? > _______________________________________________ > libav-devel mailing list > libav-devel@libav.org > https://lists.libav.org/mailman/listinfo/libav-devel _______________________________________________ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel