diff options
author | Mark Harris <mark.hsj@gmail.com> | 2016-12-15 16:44:57 -0800 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2016-12-16 03:00:03 +0100 |
commit | a5cf600ccb7f20b65d552fef5c5d010513fcf74c (patch) | |
tree | 5adb2376f08c91087b7da031d9787a05914ce746 /libavformat | |
parent | d4c1cc2b876af1e8f1a8ca258bf092b3baa31245 (diff) | |
download | ffmpeg-a5cf600ccb7f20b65d552fef5c5d010513fcf74c.tar.gz |
avformat/wavdec: Eliminate goto for clang -O0 DCE
Clang is not able to eliminate the reference to ff_spdif_probe() when
there is a goto target in the same block and optimization is disabled.
This fixes the following build failure on OS X:
./configure --disable-everything --disable-doc \
--enable-decoder=pcm_s16le --enable-demuxer=wav \
--enable-protocol=file --disable-optimizations --cc=clang
make
...
Undefined symbols for architecture x86_64:
"_ff_spdif_probe", referenced from:
_set_spdif in libavformat.a(wavdec.o)
ld: symbol(s) not found for architecture x86_64
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavformat')
-rw-r--r-- | libavformat/wavdec.c | 38 |
1 files changed, 17 insertions, 21 deletions
diff --git a/libavformat/wavdec.c b/libavformat/wavdec.c index 7176cd6f2d..ae42a6167f 100644 --- a/libavformat/wavdec.c +++ b/libavformat/wavdec.c @@ -64,34 +64,30 @@ static void set_spdif(AVFormatContext *s, WAVDemuxContext *wav) { if (CONFIG_SPDIF_DEMUXER && s->streams[0]->codecpar->codec_tag == 1) { enum AVCodecID codec; - uint8_t *buf = NULL; int len = 1<<16; int ret = ffio_ensure_seekback(s->pb, len); - int64_t pos = avio_tell(s->pb); - if (ret < 0) - goto end; - - buf = av_malloc(len); - if (!buf) { - ret = AVERROR(ENOMEM); - goto end; + if (ret >= 0) { + uint8_t *buf = av_malloc(len); + if (!buf) { + ret = AVERROR(ENOMEM); + } else { + int64_t pos = avio_tell(s->pb); + len = ret = avio_read(s->pb, buf, len); + if (len >= 0) { + ret = ff_spdif_probe(buf, len, &codec); + if (ret > AVPROBE_SCORE_EXTENSION) { + s->streams[0]->codecpar->codec_id = codec; + wav->spdif = 1; + } + } + avio_seek(s->pb, pos, SEEK_SET); + av_free(buf); + } } - len = ret = avio_read(s->pb, buf, len); - if (ret < 0) - goto end; - - ret = ff_spdif_probe(buf, len, &codec); - if (ret > AVPROBE_SCORE_EXTENSION) { - s->streams[0]->codecpar->codec_id = codec; - wav->spdif = 1; - } -end: - avio_seek(s->pb, pos, SEEK_SET); if (ret < 0) av_log(s, AV_LOG_WARNING, "Cannot check for SPDIF\n"); - av_free(buf); } } |