Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-06 Thread James Almer
On 4/3/2017 4:29 AM, Steve Lhomme wrote: > As found in HEVC. > > I put the code in mastering_display_metadata as they usually go together in > Blu-Ray UHD sources. > > -- > update the previous patch: > - rename CEA 861.3 to CTA-861.3 > - use MaxCLL and MaxFALL names that are more commonly found

[FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-03 Thread Steve Lhomme
As found in HEVC. I put the code in mastering_display_metadata as they usually go together in Blu-Ray UHD sources. -- update the previous patch: - rename CEA 861.3 to CTA-861.3 - use MaxCLL and MaxFALL names that are more commonly found - use unsigned integer rather than rational numbers as

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-02 Thread James Almer
On 4/2/2017 3:36 PM, Moritz Barsnick wrote: > On Sun, Apr 02, 2017 at 10:08:23 +0200, Steve Lhomme wrote: >> As found in HEVC. >> >> I put the code in mastering_display_metadata as they usually go together in >> Blu-Ray UHD sources. > > These spelling remarks of mine remain valid: >

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-02 Thread Moritz Barsnick
On Sun, Apr 02, 2017 at 10:08:23 +0200, Steve Lhomme wrote: > As found in HEVC. > > I put the code in mastering_display_metadata as they usually go together in > Blu-Ray UHD sources. These spelling remarks of mine remain valid: http://ffmpeg.org/pipermail/ffmpeg-devel/2017-April/209567.html

[FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-02 Thread Steve Lhomme
As found in HEVC. I put the code in mastering_display_metadata as they usually go together in Blu-Ray UHD sources. -- update the previous patch: - rename CEA 861.3 to CTA-861.3 - use MaxCLL and MaxFALL names that are more commonly found - use unsigned integer rather than rational numbers as

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-04-01 Thread Moritz Barsnick
On Thu, Mar 30, 2017 at 16:31:47 +0200, Steve Lhomme wrote: > + * Content light level (based on CEA 861.3). This payload containts data > in ^ contains > + * Max avereage light level per picture (cd/m^2). ^

[FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-03-31 Thread Steve Lhomme
As found in HEVC. I put the code in mastering_display_metadata as they usually go together in Blu-Ray UHD sources. -- update the previous patch: - rename CEA 861.3 to CTA-861.3 - use MaxCLL and MaxFALL names that are more commonly found - use unsigned integer rather than rational numbers as

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-03-31 Thread Steve Lhomme
On Thu, Mar 30, 2017 at 4:46 PM, Hendrik Leppkes wrote: > On Thu, Mar 30, 2017 at 4:31 PM, Steve Lhomme wrote: >> As found in HEVC. >> >> I put the code in mastering_display_metadata as they usually go together in >> Blu-Ray UHD sources. >> --- >>

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-03-31 Thread Steve Lhomme
On Thu, Mar 30, 2017 at 5:50 PM, James Almer wrote: > On 3/30/2017 11:31 AM, Steve Lhomme wrote: >> As found in HEVC. >> >> I put the code in mastering_display_metadata as they usually go together in >> Blu-Ray UHD sources. >> --- >> libavutil/frame.c | 1

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-03-30 Thread James Almer
On 3/30/2017 11:31 AM, Steve Lhomme wrote: > As found in HEVC. > > I put the code in mastering_display_metadata as they usually go together in > Blu-Ray UHD sources. > --- > libavutil/frame.c | 1 + > libavutil/frame.h | 6 ++ >

Re: [FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-03-30 Thread Hendrik Leppkes
On Thu, Mar 30, 2017 at 4:31 PM, Steve Lhomme wrote: > As found in HEVC. > > I put the code in mastering_display_metadata as they usually go together in > Blu-Ray UHD sources. > --- > libavutil/frame.c | 1 + > libavutil/frame.h | 6

[FFmpeg-devel] [PATCH 1/3] lavu: add support for Content Light Level side metadata

2017-03-30 Thread Steve Lhomme
As found in HEVC. I put the code in mastering_display_metadata as they usually go together in Blu-Ray UHD sources. --- libavutil/frame.c | 1 + libavutil/frame.h | 6 ++ libavutil/mastering_display_metadata.c | 18