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

Nicolas Gaullier nicolas.gaullier at arkena.com
Tue Aug 6 18:08:11 EEST 2019


Corrected with tomas's feedback

The guard band minimum size is now specified in bytes to simplify and restrict the test to an integer checking,
it is no big deal for me, and anyway I have set it to 0 by default, just overridable if someone wants to set it.

S337M_MAX_RECOMMENDED_PROBE_SIZE is now prefixed with AVPRIV_, it is currently used for the wav demux, and could be helpful
anyware else s337m is implemented. I don't feel other macros would be helpful to share; for example, DOLBY_E_PHASE_MIN/MAX represent
some minimal core safeguards for logging alarms, but the accurate min/max values that would be really relevant and worth sharing
depend on video resolution...

Nicolas Gaullier (6):
  avformat/s337m: Use base AVClass for av_log usage
  avformat/s337m: Split read_packet/get_packet
  avformat/s337m: Make available as subdemuxer
  avformat/wavdec: s337m support
  avformat/wavdec: reindent
  avformat/s337m: Test wav subdemux

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

-- 
2.14.1.windows.1



More information about the ffmpeg-devel mailing list