[FFmpeg-devel] [PATCH v1] avcodec/v210enc: add yuv420p/yuv420p10 input pixel format support

Limin Wang lance.lmwang at gmail.com
Mon Sep 23 04:09:48 EEST 2019


On Sun, Sep 22, 2019 at 07:15:53PM +0100, Derek Buitenhuis wrote:
> On 22/09/2019 17:18, Kieran Kunhya wrote:
> > And just like in the kernel, zero copy technology is maintained outside.
> > Good design is separation of different functionality, otherwise the
> > combinations of codepaths is exponentiated.
> 
> Agreed with everyone else here, this code doesn't belong in an encoder.
> 
> libavcodec is a general use codec library, and this doesn't fit that
> design or philosophy at all. It's hideous.

I agree with your guys. Can I submit a patch to remove AV_PIX_FMT_YUV422P, it's not belong
to v210 encoder also?  It make me misunderstanding it's acceptable to add other format.


> 
> - Derek
> _______________________________________________
> 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