[FFmpeg-devel] [PATCH 2/2] avfilter/vf_scale: allow overriding in/out yuv colorspace type

Paul B Mahol onemda at gmail.com
Mon Jul 15 21:31:58 CEST 2013


On 7/15/13, Reimar Doeffinger <Reimar.Doeffinger at gmx.de> wrote:
> On Mon, Jul 15, 2013 at 01:19:34AM +0200, Michael Niedermayer wrote:
>> > That is not exacly same, as qcif/cif/ .. are not listed at all.
>> >
>> > Also it seems little akward to me that in same option one can give
>> > custom matrix and already available one.
>>
>> yes
>>
>> Do people want me to change the patch somehow or are they ok with it
>> as it is. I dont mind changing it but such request must come with
>> understanding of the sideeffects
>
> Not that I really mind, but if we have multiple options to set the
> same thing how do we decide which to use? E.g. if the user set both?

Well, if type is set to custom then matrix is read.
If matrix is set but type is not custom, warning is displayed.

I do not insist on anything, you can commit whatever you like.

> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list