Re: [FFmpeg-devel] [PATCH] colorspace: Rename jedec-p22 to ebu3213

2019-08-09 Thread Kevin Wheatley
On Fri, Aug 9, 2019 at 12:40 PM Hendrik Leppkes wrote: > The enum and our values are aligned to the H.273 / ISO/IEC 23001-8 > standards, which documents this entry to correspond to the primaries > in use by vf_colorspace That makes sense, although I'm now interested to find out where those

Re: [FFmpeg-devel] [PATCH] colorspace: Rename jedec-p22 to ebu3213

2019-08-09 Thread Hendrik Leppkes
On Fri, Aug 9, 2019 at 12:32 PM Kevin Wheatley wrote: > > Is there a change to include the EBU primaries? > > https://tech.ebu.ch/docs/tech/tech3213.pdf > > White 0.313 0.329 > Red 0.64 0.33 > Green 0.29 0.60 > Blue 0.15 0.06 > > as the ones currently called AVCOL_PRI_JEDEC_P22 are not those ones

Re: [FFmpeg-devel] [PATCH] colorspace: Rename jedec-p22 to ebu3213

2019-08-09 Thread Kevin Wheatley
Is there a change to include the EBU primaries? https://tech.ebu.ch/docs/tech/tech3213.pdf White 0.313 0.329 Red 0.64 0.33 Green 0.29 0.60 Blue 0.15 0.06 as the ones currently called AVCOL_PRI_JEDEC_P22 are not those ones at least in vf_colorspace.c [AVCOL_PRI_JEDEC_P22] = { WP_D65, {

Re: [FFmpeg-devel] [PATCH] colorspace: Rename jedec-p22 to ebu3213

2019-08-08 Thread James Almer
On 8/8/2019 9:01 PM, rzu...@tebako.net wrote: > From: Raphaël Zumer > > Internally, this adds an EBU3213 alias to JEDEC_P22, > and changes the name string to match ITU-T H.273. > --- > libavcodec/options_table.h | 2 +- > libavfilter/vf_colorspace.c | 2 +- > libavfilter/vf_setparams.c | 2 +-

[FFmpeg-devel] [PATCH] colorspace: Rename jedec-p22 to ebu3213

2019-08-08 Thread rzumer
From: Raphaël Zumer Internally, this adds an EBU3213 alias to JEDEC_P22, and changes the name string to match ITU-T H.273. --- libavcodec/options_table.h | 2 +- libavfilter/vf_colorspace.c | 2 +- libavfilter/vf_setparams.c | 2 +- libavfilter/vf_zscale.c | 2 +- libavutil/pixdesc.c