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

[Anton Khirnov]

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.

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.

-- 
Anton Khirnov