[FFmpeg-devel] [PATCH 0/9] avformat: Use s337m subdemux inside wav

Nicolas Gaullier nicolas.gaullier at cji.paris
Fri Jan 3 17:56:27 EET 2020


This is a new version of my previous patchset reviewed by Tomas.
http://ffmpeg.org/pipermail/ffmpeg-devel/2019-August/247677.html

It takes into account the last feedback from Carl Eugen,
I mean that the integration of s337m in wav is now similar to the existing one for spdif,
main differences remains that :
- s337m is not enabled by default (activated by the 'dolbyeprobe' AVOption)
- container bit resolution is checked (DolbyE stream bit resolution = WAV PCM bit resolution)
- takes data_end into account (see my latest patch here below: 'Fix s337m last packet parsing'),
but this one could also be 'forwarded' to spdif
- verbose detection and phase (I think it is a broadcast requirement)

Fate samples can be found here:
- 512.wav : http://0x0.st/zdW-.wav
- 512.mxf.pcm : http://0x0.st/zdWo.pcm

Thank you for this new review...

Nicolas Gaullier (9):
  avformat/s337m: Use base AVClass for av_log usage
  avformat/s337m: Split read_packet/get_packet
  avformat/s337m: Consider container bit resolution
  avformat/s337m: New ff_s337m_probe()
  avformat/wavdec: s337m support
  avformat/wavdec: fix s337m/spdif probing beyond data_end
  avformat/wavdec: Fix s337m last packet parsing
  avformat/wavdec: Reindent after last commit
  avformat/wavdec: Test s337m

 libavformat/s337m.c  | 87 +++++++++++++++++++++++++++++++++++++++++++---------
 libavformat/s337m.h  | 58 +++++++++++++++++++++++++++++++++++
 libavformat/wavdec.c | 43 ++++++++++++++++++--------
 tests/Makefile       |  1 +
 tests/fate-run.sh    |  4 +++
 tests/fate/audio.mak |  5 +++
 6 files changed, 171 insertions(+), 27 deletions(-)
 create mode 100644 libavformat/s337m.h

-- 
2.14.1.windows.1



More information about the ffmpeg-devel mailing list