[FFmpeg-devel] [PATCH]lavf/amr: Add amrnb and amrwb demuxers
Carl Eugen Hoyos
ceffmpeg at gmail.com
Tue Oct 3 00:47:57 EEST 2017
2017-10-02 23:02 GMT+02:00 Michael Niedermayer <michael at niedermayer.cc>:
> On Sun, Oct 01, 2017 at 06:23:50PM +0200, Carl Eugen Hoyos wrote:
>> 2017-09-27 18:08 GMT+02:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>> > The existing amr demuxer does not allow reading streams,
>> > it requires the 3GPP-conforming file header.
>> > Attached patch allows reading amrnb and amrwb from (live)
>> > streams, fixes ticket #6678.
>> New patch with auto-detection attached, passes probecheck.
>> Please comment, Carl Eugen
> breaks mingw64
> In file included from src/libavformat/amrnb.c:26:0:
> src/libavcodec/amrnbdata.h:50:5: error: expected identifier before ‘(’ token
> NO_DATA = 15 ///< no transmission
> make: *** [libavformat/amrnb.o] Error 1
> make: *** Waiting for unfinished jobs....
> STRIP libavfilter/x86/vf_yadif.o
> STRIP libavfilter/x86/vf_removegrain.o
> In file included from src/libavformat/amrwb.c:26:0:
> src/libavcodec/amrwbdata.h:64:5: error: expected identifier before ‘(’ token
> NO_DATA ///< no transmission
> make: *** [libavformat/amrwb.o] Error 1
The errors are apparently caused by the following definitions in a mingw header:
#define WSANO_DATA (WSABASEERR + 1004)
#define NO_DATA WSANO_DATA
Do you know why this does not break the compilation of
the amr decoders?
Thank you, Carl Eugen
More information about the ffmpeg-devel