[FFmpeg-devel] [PATCH] libavcodec/options_table: Add missing colorspace options

[Harry Mallon]

> On 8 Sep 2020, at 14:29, Gyan Doshi <ffmpeg at gyani.pro> wrote:
> 
> On 08-09-2020 06:23 pm, Harry Mallon wrote:
>> 
>> 
>>> On 8 Sep 2020, at 13:08, Gyan Doshi <ffmpeg at gyani.pro> wrote:
>>> 
>>> Hi Harry,
>>> 
>>> On 06-09-2020 04:24 pm, Gyan Doshi wrote:
>>>> 
>>>> On 06-09-2020 02:59 pm, Harry Mallon wrote:
>>>>> Is there anything I need to do to move this forwards? I think it is relatively uncontroversial?
>>>> Agreed. Will push in 3 days if there are no objections.
>>> Can you link to some doc references for the entries you added?
>>> 
>> I'm not 100% sure what you mean, do you mean add some links to the doc/codecs.texi file, or just for the mailing list? ICtCp is definitely in ITU-R Rec 2100, I'm not sure about the others. It just seemed to me that they were missing from the command line arguments (and I needed to manually set ICtCp for something).
> 
> For the ML, at present. Ideally, the canonical standard docs. Avoid Wikipedia, forum posts..etc
> 

Well ICtCp (which is the one I am interested in) is in ITU-R BT.2100 https://www.itu.int/dms_pubrec/itu-r/rec/bt/R-REC-BT.2100-2-201807-I!!PDF-E.pdf
I am not sure where the chroma-derived ones come from. They are all already supported in ffmpeg (https://github.com/FFmpeg/FFmpeg/commit/f3571048669bf876681499f49e9df492f05f73c6), just they were missing from this specific command line option.

Best,
Harry