mbox series

[FFmpeg-devel,00/11,v6] AVCodecContext and AVCodecParameters side data

Message ID 20231004122849.56604-1-jamrial@gmail.com
Headers show
Series AVCodecContext and AVCodecParameters side data | expand

Message

James Almer Oct. 4, 2023, 12:28 p.m. UTC 
Changes since the previous version:
- Zeroed padding to side data allocated by av_packet_side_data_new() (Patch 1).
- Renamed AVCodecParameters.{nb,}side_data to {nb,}coded_side_data, to both
  match the name in AVCodecContext and keep the side_data name free for future
  use (Patch 2).
- Rewritten documentation to AVPacketSideData (Patch 4)
- Display Matrix global side data is now propagated to output frames (Patch 9).

Will add version bumps and APIChanges entries and push the set later this week
unless there are objections.

James Almer (11):
  avcodec/packet: add generic side data helpers
  avcodec/codec_par: add side data to AVCodecParameters
  avformat/avformat: use the side data from AVStream.codecpar
  avcodec/packet: add some documentation for AVPacketSideData
  fftools/ffmpeg: stop using AVStream.side_data
  fftools/ffplay: stop using AVStream.side_data
  fftools/ffprobe: stop using AVStream.side_data
  avcodec/hevcdec: check for DOVI configuration record in AVCodecContext
    side data
  avcodec/decode: propagate global side data to frames
  fftools/ffmpeg: stop injecting stream side data in packets
  fftools/ffplay: stop injecting stream side data in packets

 fftools/ffmpeg_demux.c                        |  31 +--
 fftools/ffmpeg_enc.c                          |  32 +--
 fftools/ffmpeg_filter.c                       |   7 +-
 fftools/ffmpeg_mux_init.c                     |  24 ++-
 fftools/ffplay.c                              |  11 +-
 fftools/ffprobe.c                             |  30 ++-
 libavcodec/avcodec.h                          |   2 +-
 libavcodec/avpacket.c                         | 101 +++++++++
 libavcodec/codec_par.c                        |  48 +++++
 libavcodec/codec_par.h                        |  11 +
 libavcodec/decode.c                           |  60 +++++-
 libavcodec/decode.h                           |   6 +
 libavcodec/hevcdec.c                          |  15 +-
 libavcodec/packet.h                           | 115 ++++++++++-
 libavdevice/android_camera.c                  |  10 +-
 libavformat/avformat.c                        |  42 +---
 libavformat/avformat.h                        |  28 ++-
 libavformat/concatdec.c                       |   1 -
 libavformat/dashdec.c                         |  11 -
 libavformat/demux.c                           |  53 +++--
 libavformat/demux_utils.c                     |   4 +
 libavformat/dovi_isom.c                       |   8 +-
 libavformat/dump.c                            |   6 +-
 libavformat/hls.c                             |  11 -
 libavformat/hlsenc.c                          |  11 +-
 libavformat/internal.h                        |   4 +
 libavformat/matroskadec.c                     |  47 +++--
 libavformat/matroskaenc.c                     |  55 +++--
 libavformat/mov.c                             |  92 +++++----
 libavformat/movenc.c                          |  94 +++++----
 libavformat/mp3enc.c                          |  10 +-
 libavformat/mpegenc.c                         |  22 +-
 libavformat/mpegts.c                          |  10 +-
 libavformat/mux.c                             |  21 ++
 libavformat/mxfdec.c                          |  25 ++-
 libavformat/mxfenc.c                          |  12 +-
 libavformat/options.c                         |   2 +
 libavformat/replaygain.c                      |  10 +-
 libavformat/seek.c                            |   2 +
 libavformat/version_major.h                   |   1 +
 tests/ref/fate/autorotate                     |   4 +-
 tests/ref/fate/copy-trac3074                  |   2 +-
 tests/ref/fate/hevc-dv-rpu                    |  18 ++
 tests/ref/fate/matroska-avoid-negative-ts     |   2 +-
 tests/ref/fate/matroska-dovi-write-config7    |   2 +-
 tests/ref/fate/matroska-dovi-write-config8    |   2 +-
 tests/ref/fate/matroska-encoding-delay        |   2 +-
 .../fate/matroska-mastering-display-metadata  |   4 +-
 tests/ref/fate/matroska-spherical-mono-remux  |   4 +-
 tests/ref/fate/matroska-stereo_mode           |   8 +-
 tests/ref/fate/matroska-vp8-alpha-remux       |   2 +-
 .../ref/fate/mov-mp4-disposition-mpegts-remux |   4 +-
 tests/ref/fate/mov-zombie                     | 195 ++++++++++++------
 tests/ref/fate/mxf-d10-user-comments          |   2 +-
 tests/ref/fate/mxf-remux-applehdr10           |   2 +-
 tests/ref/fate/vp9-superframe-bsf             |   2 +-
 56 files changed, 909 insertions(+), 431 deletions(-)

Comments

Anton Khirnov Oct. 4, 2023, 2:47 p.m. UTC | #1 
Quoting James Almer (2023-10-04 14:28:38)
> Changes since the previous version:
> - Zeroed padding to side data allocated by av_packet_side_data_new() (Patch 1).
> - Renamed AVCodecParameters.{nb,}side_data to {nb,}coded_side_data, to both
>   match the name in AVCodecContext and keep the side_data name free for future
>   use (Patch 2).
> - Rewritten documentation to AVPacketSideData (Patch 4)
> - Display Matrix global side data is now propagated to output frames (Patch 9).
> 
> Will add version bumps and APIChanges entries and push the set later this week
> unless there are objections.

\o/

Very nice