Merge commit '94c54d97e7f4fe90570c323803f2bdf6246c1010'
authorJames Almer <jamrial@gmail.com>
Mon, 30 Oct 2017 19:52:36 +0000 (16:52 -0300)
committerJames Almer <jamrial@gmail.com>
Mon, 30 Oct 2017 19:52:36 +0000 (16:52 -0300)
* commit '94c54d97e7f4fe90570c323803f2bdf6246c1010':
  mlp: Factor out channel layout subset checks

Merged-by: James Almer <jamrial@gmail.com>
1  2 
libavcodec/mlp_parser.c
libavcodec/mlp_parser.h
libavcodec/mlpdec.c

@@@ -339,11 -334,13 +344,13 @@@ static int mlp_parse(AVCodecParserConte
              goto lost_sync;
          }
      } else {
 -        BitstreamContext bc;
 +        GetBitContext gb;
          MLPHeaderInfo mh;
+         int stereo_requested = ff_mlp_channel_layout_subset(avctx->request_channel_layout,
+                                                             AV_CH_LAYOUT_STEREO);
  
 -        bitstream_init8(&bc, buf + 4, buf_size - 4);
 -        if (ff_mlp_read_major_sync(avctx, &mh, &bc) < 0)
 +        init_get_bits(&gb, buf + 4, (buf_size - 4) << 3);
 +        if (ff_mlp_read_major_sync(avctx, &mh, &gb) < 0)
              goto lost_sync;
  
          avctx->bits_per_raw_sample = mh.group1_bits;
          avctx->sample_rate = mh.group1_samplerate;
          s->duration = mh.access_unit_size;
  
 +        if(!avctx->channels || !avctx->channel_layout) {
          if (mh.stream_type == 0xbb) {
              /* MLP stream */
-             if (avctx->request_channel_layout &&
-                 (avctx->request_channel_layout & AV_CH_LAYOUT_STEREO) ==
-                 avctx->request_channel_layout &&
-                 mh.num_substreams > 1) {
+             if (stereo_requested && mh.num_substreams > 1) {
                  avctx->channels       = 2;
                  avctx->channel_layout = AV_CH_LAYOUT_STEREO;
              } else {
@@@ -63,9 -61,8 +63,11 @@@ typedef struct MLPHeaderInf
  } MLPHeaderInfo;
  
  
 -int ff_mlp_read_major_sync(void *log, MLPHeaderInfo *mh, BitstreamContext *bc);
 +int ff_mlp_read_major_sync(void *log, MLPHeaderInfo *mh, GetBitContext *gb);
 +uint64_t ff_truehd_layout(int chanmap);
 +
 +extern const uint64_t ff_mlp_layout[32];
  
+ int ff_mlp_channel_layout_subset(uint64_t channel_layout, uint64_t mask);
  #endif /* AVCODEC_MLP_PARSER_H */
Simple merge