[FFmpeg-devel] [PATCH 0/1] avformat/dolby_e: Add a parser
Nicolas Gaullier
nicolas.gaullier at cji.paris
Mon Jan 11 19:37:02 EET 2021
Note: my first intend here is, afterwards, to provide s377m support to the wav demuxer,
(the actual code leads to pts issues : I sent a workaround
https://patchwork.ffmpeg.org/project/ffmpeg/patch/20201215171324.99-1-nicolas.gaullier@cji.paris/
that was rejected, so this new work for a Dolby E parser is simply a follow up).
I have tested that this patch actually fixes PTS generation when demuxing s337m wav samples.
This is a first try, so, yes, I kown it is missing Changelog and version updates. I think the review
will take some time so this could be updated lastly.
This is a single patch, I think it is easier for review anyway, but I expect you will ask me to split it in several files like
- dolby_e_parser.c/.h
- dolby_edec.c/.h
Anyway, if so, I think I would keep this first patch as is and simply add a new patch to just split all the things.
Unless you think it is better to split the parser code from the very beginning/first patch.
So, I keep listening about how this work should be presented, in term of number of patches etc.
Thank you in advance for the review
Nicolas
Nicolas Gaullier (1):
avformat/dolby_e: Add a parser
libavcodec/dolby_e.c | 260 ++++++++++++++++++++++++++++---------------
libavcodec/dolby_e.h | 38 ++++++-
libavcodec/parsers.c | 1 +
3 files changed, 202 insertions(+), 97 deletions(-)
--
2.27.0.windows.1
More information about the ffmpeg-devel
mailing list