diff options
author | wm4 <wm4@nowhere> | 2015-01-30 14:27:43 +0100 |
---|---|---|
committer | Diogo Franco (Kovensky) <diogomfranco@gmail.com> | 2015-02-03 14:36:36 +0900 |
commit | e8df9ec4b77530ea6311ec847886dd82e798759e (patch) | |
tree | 329ea5aebf7d192d48a721fe4a305c2b887e1f64 | |
parent | 57e57429ea6e57d4f38d6d612d85e8a11c78aa21 (diff) | |
download | mpv-e8df9ec4b77530ea6311ec847886dd82e798759e.tar.bz2 mpv-e8df9ec4b77530ea6311ec847886dd82e798759e.tar.xz |
av_common: add comment about using now-deprecated libavcodec field
FFmpeg and Libav have the stupid practice of replacing and deprecating
API symbols on the same day. So with FFmpeg git, this is useless and
will print a compile time warning, while it's required with all stable
releases, and might lead to decoding errors with xvid/avi (apparently).
Add a comment before someone writes a patch and I have to explain it all
over again.
-rw-r--r-- | common/av_common.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/common/av_common.c b/common/av_common.c index 2d976bbc8d..bc2845d4d4 100644 --- a/common/av_common.c +++ b/common/av_common.c @@ -55,7 +55,6 @@ void mp_copy_lav_codec_headers(AVCodecContext *avctx, AVCodecContext *st) { mp_lavc_set_extradata(avctx, st->extradata, st->extradata_size); avctx->codec_tag = st->codec_tag; - avctx->stream_codec_tag = st->stream_codec_tag; avctx->bit_rate = st->bit_rate; avctx->width = st->width; avctx->height = st->height; @@ -67,6 +66,8 @@ void mp_copy_lav_codec_headers(AVCodecContext *avctx, AVCodecContext *st) avctx->block_align = st->block_align; avctx->channel_layout = st->channel_layout; avctx->bits_per_coded_sample = st->bits_per_coded_sample; + // Required in FFmpeg 2.5.x / Libav 11, deprecated afterwards. + avctx->stream_codec_tag = st->stream_codec_tag; } // We merely pass-through our PTS/DTS as an int64_t; libavcodec won't use it. |