Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-12 Thread myp...@gmail.com
On Sun, Apr 12, 2020 at 9:06 PM Jean-Baptiste Kempf wrote: > > > > On Sun, Apr 12, 2020, at 03:22, myp...@gmail.com wrote: > > On Sat, Apr 11, 2020 at 10:18 PM Jean-Baptiste Kempf > > wrote: > > > > > > I really do not think it is a good idea to have a header with the name > > > dolby in it. >

Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-12 Thread Jean-Baptiste Kempf
On Sun, Apr 12, 2020, at 03:22, myp...@gmail.com wrote: > On Sat, Apr 11, 2020 at 10:18 PM Jean-Baptiste Kempf > wrote: > > > > I really do not think it is a good idea to have a header with the name > > dolby in it. > > > Can you give some details, I don't why is a bad idea. Dolby is a tradem

Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-11 Thread myp...@gmail.com
On Sat, Apr 11, 2020 at 11:01 PM Jan Ekström wrote: > > On Sat, Apr 11, 2020 at 5:49 PM Carl Eugen Hoyos wrote: > > > > Am Sa., 11. Apr. 2020 um 16:08 Uhr schrieb Jun Zhao : > > > > > > From: vacinfang > > > > > > add dolby vision related header > > > > Apart from the name of the header file: >

Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-11 Thread myp...@gmail.com
On Sat, Apr 11, 2020 at 10:18 PM Jean-Baptiste Kempf wrote: > > I really do not think it is a good idea to have a header with the name dolby > in it. > Can you give some details, I don't why is a bad idea. > On Sat, Apr 11, 2020, at 15:44, Jun Zhao wrote: > > From: vacinfang > > > > add dolby vi

Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-11 Thread Jan Ekström
On Sat, Apr 11, 2020 at 5:49 PM Carl Eugen Hoyos wrote: > > Am Sa., 11. Apr. 2020 um 16:08 Uhr schrieb Jun Zhao : > > > > From: vacinfang > > > > add dolby vision related header > > Apart from the name of the header file: > Could you explain why it is needed? > > Is the Dolby Vision metadata not

Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-11 Thread Carl Eugen Hoyos
Am Sa., 11. Apr. 2020 um 16:08 Uhr schrieb Jun Zhao : > > From: vacinfang > > add dolby vision related header Apart from the name of the header file: Could you explain why it is needed? Is the Dolby Vision metadata not a black box that libavformat has to provide to interested decoders? Carl Eug

Re: [FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-11 Thread Jean-Baptiste Kempf
I really do not think it is a good idea to have a header with the name dolby in it. On Sat, Apr 11, 2020, at 15:44, Jun Zhao wrote: > From: vacinfang > > add dolby vision related header > > Signed-off-by: vacinfang > --- > libavutil/Makefile| 1 + > libavutil/dolby_vision_meta.h

[FFmpeg-devel] [PATCH v2 2/7] lavutil: add dolby vision related header

2020-04-11 Thread Jun Zhao
From: vacinfang add dolby vision related header Signed-off-by: vacinfang --- libavutil/Makefile| 1 + libavutil/dolby_vision_meta.h | 58 +++ 2 files changed, 59 insertions(+) create mode 100644 libavutil/dolby_vision_meta.h diff --git a/l