Re: [FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-09-01 Thread James Almer
On 8/31/2019 10:30 PM, Raphaël Zumer wrote: > It's been about 3 weeks, so sending a reminder in case anyone can review > this. Patchset pushed, thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-08-31 Thread Raphaël Zumer
It's been about 3 weeks, so sending a reminder in case anyone can review this. On 2019-08-11 9:54 a.m., rzu...@tebako.net wrote: From: Raphaël Zumer This is an alias for JEDEC P22. The name associated with the value is also changed from "jedec-p22" to "ebu3213" to match ITU-T H.273.

Re: [FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-08-17 Thread Raphaël Zumer
I would like to mention that this set of patches is revised following previous comments. Since I did not set the version number correctly, it might have slipped past some people's radars. If there are any further changes needed, please let me know. RZ On 2019-08-11 9:54 a.m.,

[FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-08-11 Thread rzumer
From: Raphaël Zumer This is an alias for JEDEC P22. The name associated with the value is also changed from "jedec-p22" to "ebu3213" to match ITU-T H.273. Signed-off-by: Raphaël Zumer --- doc/APIchanges | 3 +++ libavutil/pixdesc.c | 2 +- libavutil/pixfmt.h | 3 ++-

Re: [FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-08-09 Thread Raphaël Zumer
On 2019-08-09 7:51 a.m., Vittorio Giovara wrote: this value was already present, this is just a rename I think it counts as a new enum value (variant), even though it aliases another. I initially wrote "rename", but reworded the commit message based on James Almer's comment: The subject

Re: [FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-08-09 Thread Vittorio Giovara
On Fri, Aug 9, 2019 at 3:29 AM wrote: > From: Raphaël Zumer > > This is an alias for JEDEC P22. > > The name associated with the value is also changed > from "jedec-p22" to "ebu3213" to match ITU-T H.273. > > Signed-off-by: Raphaël Zumer > --- > doc/APIchanges | 3 +++ >

[FFmpeg-devel] [PATCH 1/3] avutil/pixfmt: Add EBU Tech. 3213-E AVColorPrimaries value

2019-08-08 Thread rzumer
From: Raphaël Zumer This is an alias for JEDEC P22. The name associated with the value is also changed from "jedec-p22" to "ebu3213" to match ITU-T H.273. Signed-off-by: Raphaël Zumer --- doc/APIchanges | 3 +++ libavutil/pixdesc.c | 2 +- libavutil/pixfmt.h | 1 + libavutil/version.h