[FFmpeg-devel] [PATCH v6 0/9] Support Dolby Vision

[mypopy at gmail.com]

On Thu, Apr 23, 2020 at 5:55 PM Anton Khirnov <anton at khirnov.net> wrote:
>
> Quoting mypopy at gmail.com (2020-04-23 04:16:41)
> > On Wed, Apr 22, 2020 at 7:53 PM Jun Zhao <mypopydev at gmail.com> wrote:
> > >
> > > From: Jun Zhao <barryjzhaotencent.com>
> > >
> > > V6:
> > > - split the header and .c file
> > > - bump version and update APIChnage
> > >
> > > V5:
> > > - add a constructor for DOVI (tks Anton Khirnov)
> > > - some minor coding style clean
> > >
> > > V4:
> > > - change the DOVI dump message (tks Jan Ekström)
> > > - fix the file order in Makefile
> > >
> > > V3:
> > > - rename the file name for Dolby Vision trademark risk(tks Jean-Baptiste Kempf)
> > > - fix other typo issue
> > >
> > > V2:
> > > - used sidedata for Dolby Vision information (tks Jan Ekström's suggestion)
> > > - enabled dumping Dolby Vision sidedata for ffmpeg/ffprobe tools
> > > - enabled dvcC/dvvC box from Dolby Vision sidedata
> > >
> > >
> > > Jun Zhao (6):
> > >   lavc: add a new sidedata type for DOVI
> > >   lavf/dump: dump DOVI side data
> > >   fftools/ffprobe: support DOVI sidedata
> > >   lavf/movenc: enable dvcC/dvvC box support from DOVI sidedata
> > >   lavc/version: bump minor version for DOVI sidedata
> > >   doc/APIChanges: add AV_PKT_DATA_DOVI_CONF entry
> > >
> > > vacingfang (3):
> > >   lavutil: add DOVI related header
> > >   lavf/mpegts: support DOVI Video Stream Descriptor
> > >   lavf/mov: support dvcC/dvvC box for DOVI
> > >
> > >  doc/APIchanges        |  3 +++
> > >  fftools/ffprobe.c     | 11 ++++++++
> > >  libavcodec/avpacket.c |  1 +
> > >  libavcodec/packet.h   |  9 +++++++
> > >  libavcodec/version.h  |  2 +-
> > >  libavformat/dump.c    | 19 ++++++++++++++
> > >  libavformat/mov.c     | 60 +++++++++++++++++++++++++++++++++++++++++++
> > >  libavformat/movenc.c  | 35 ++++++++++++++++++++++++++
> > >  libavformat/mpegts.c  | 48 +++++++++++++++++++++++++++++++++++
> > >  libavutil/Makefile    |  2 ++
> > >  libavutil/dovi_meta.c | 35 ++++++++++++++++++++++++++
> > >  libavutil/dovi_meta.h | 70 +++++++++++++++++++++++++++++++++++++++++++++++++++
> > >  12 files changed, 294 insertions(+), 1 deletion(-)
> > >  create mode 100644 libavutil/dovi_meta.c
> > >  create mode 100644 libavutil/dovi_meta.h
> > >
> > Patchset pushed, tks
>
> You forgot the lavu version bump+APIchanges entry.
>
Will update this part
> Also, I think it is good form to leave at least 24 hours between posting
> a patch and pushing it, especially when it changes public API.
> Not everyone is always able to review quickly.
>
Tks the remind and comments/review again, will follow the suggestion next time.