[FFmpeg-cvslog] mlp_parser: fix request_channel_layout behavior.

Tim Walker git at videolan.org
Sun Feb 9 03:00:52 CET 2014


ffmpeg | branch: master | Tim Walker <tdskywalker at gmail.com> | Sat Feb  8 17:21:14 2014 +0000| [c1e7de8f62c7fd71e23d62969a618f042059381f] | committer: Tim Walker

mlp_parser: fix request_channel_layout behavior.

When request_channel_layout is 0,
all substreams should be decoded.

Thanks to Michael Niedermayer for spotting.

Also fix a mismatch between the parser and
decoder when request_channel_layout is a
subset of Stereo.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=c1e7de8f62c7fd71e23d62969a618f042059381f
---

 libavcodec/mlp_parser.c |   15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/libavcodec/mlp_parser.c b/libavcodec/mlp_parser.c
index a30cd05..075227f 100644
--- a/libavcodec/mlp_parser.c
+++ b/libavcodec/mlp_parser.c
@@ -335,7 +335,9 @@ FF_DISABLE_DEPRECATION_WARNINGS
             } else
 FF_ENABLE_DEPRECATION_WARNINGS
 #endif
-            if (avctx->request_channel_layout == AV_CH_LAYOUT_STEREO &&
+            if (avctx->request_channel_layout &&
+                (avctx->request_channel_layout & AV_CH_LAYOUT_STEREO) ==
+                avctx->request_channel_layout &&
                 mh.num_substreams > 1) {
                 avctx->channels       = 2;
                 avctx->channel_layout = AV_CH_LAYOUT_STEREO;
@@ -358,13 +360,16 @@ FF_DISABLE_DEPRECATION_WARNINGS
             } else
 FF_ENABLE_DEPRECATION_WARNINGS
 #endif
-            if (avctx->request_channel_layout == AV_CH_LAYOUT_STEREO &&
-                mh.num_substreams > 1) {
+                if (avctx->request_channel_layout &&
+                    (avctx->request_channel_layout & AV_CH_LAYOUT_STEREO) ==
+                    avctx->request_channel_layout &&
+                    mh.num_substreams > 1) {
                 avctx->channels       = 2;
                 avctx->channel_layout = AV_CH_LAYOUT_STEREO;
             } else if (!mh.channels_thd_stream2 ||
-                       (mh.channel_layout_thd_stream1 & avctx->request_channel_layout) ==
-                       avctx->request_channel_layout) {
+                       (avctx->request_channel_layout &&
+                        (avctx->request_channel_layout & mh.channel_layout_thd_stream1) ==
+                        avctx->request_channel_layout)) {
                 avctx->channels       = mh.channels_thd_stream1;
                 avctx->channel_layout = mh.channel_layout_thd_stream1;
             } else {



More information about the ffmpeg-cvslog mailing list