From: Misty De Meo <mistyde...@gmail.com> I think this version of the patch should address the issue. If frames can't be parsed, the original data is passed through unaltered. From what I can tell about the RIFF WAVE .at3 files, they don't use the format being parsed at all and don't require parsing. With the updated version of this patch, they play fine. The new fate atrac3p conversion test behaves the same on both master and with these patches, confirming there aren't any regressions.
Here's the media used in the tests. In response to your previous feedback, I've reduced the PSMF test files to 64KB each; that's enough for the ffprobe tests to succeed. https://public.drac.at/bgm01.at3 https://public.drac.at/bgm01.s16 https://public.drac.at/00006.MPS https://public.drac.at/EV01_01_0500D.PMF Maxim Poliakovski (1): mpeg: add experimental support for PSMF audio. Misty De Meo (5): fate: add atrac3p conversion test oma: move some constants into libavcodec Fix detecting ATRAC3 audio from MPS files mpeg: fix use of deprecated struct psmf: add FATE tests Changelog | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/atrac3plus_parser.c | 170 ++++++++++++++++++++++++++++++++++ libavformat/oma.c => libavcodec/oma.h | 27 +++--- libavcodec/version.h | 2 +- libavformat/Makefile | 4 +- libavformat/mpeg.c | 15 +++ libavformat/mpeg.h | 1 + libavformat/oma.h | 21 ++--- libavformat/omadec.c | 13 +-- libavformat/omaenc.c | 7 +- libavformat/version.h | 2 +- tests/Makefile | 1 + tests/fate/atrac.mak | 4 + tests/fate/psmf.mak | 23 +++++ 16 files changed, 254 insertions(+), 39 deletions(-) create mode 100644 libavcodec/atrac3plus_parser.c rename libavformat/oma.c => libavcodec/oma.h (65%) create mode 100644 tests/fate/psmf.mak -- 2.15.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
