Merge commit '831018b0bbe26a603802a9022472f714a59293be'
authorJames Almer <jamrial@gmail.com>
Mon, 30 Oct 2017 20:47:21 +0000 (17:47 -0300)
committerJames Almer <jamrial@gmail.com>
Mon, 30 Oct 2017 20:47:21 +0000 (17:47 -0300)
* commit '831018b0bbe26a603802a9022472f714a59293be':
  mpeg4audio: Make avpriv_copy_pce_data() inline

Merged-by: James Almer <jamrial@gmail.com>
1  2 
libavcodec/aac_adtstoasc_bsf.c
libavcodec/mpeg4audio.c
libavcodec/mpeg4audio.h
libavformat/adtsenc.c
libavformat/latmenc.c

Simple merge
@@@ -149,61 -143,5 +149,21 @@@ int ff_mpeg4audio_get_config_gb(MPEG4Au
      if ((c->ps == -1 && c->object_type != AOT_AAC_LC) || c->channels & ~0x01)
          c->ps = 0;
  
 -    return specific_config_bitindex;
 +    return specific_config_bitindex - start_bit_index;
 +}
 +
 +int avpriv_mpeg4audio_get_config(MPEG4AudioConfig *c, const uint8_t *buf,
 +                                 int bit_size, int sync_extension)
 +{
 +    GetBitContext gb;
 +    int ret;
 +
 +    if (bit_size <= 0)
 +        return AVERROR_INVALIDDATA;
 +
 +    ret = init_get_bits(&gb, buf, bit_size);
 +    if (ret < 0)
 +        return ret;
 +
 +    return ff_mpeg4audio_get_config_gb(c, &gb, sync_extension);
  }
- static av_always_inline unsigned int copy_bits(PutBitContext *pb,
-                                                GetBitContext *gb,
-                                                int bits)
- {
-     unsigned int el = get_bits(gb, bits);
-     put_bits(pb, bits, el);
-     return el;
- }
- int avpriv_copy_pce_data(PutBitContext *pb, GetBitContext *gb)
- {
-     int five_bit_ch, four_bit_ch, comment_size, bits;
-     int offset = put_bits_count(pb);
-     copy_bits(pb, gb, 10);                  //Tag, Object Type, Frequency
-     five_bit_ch  = copy_bits(pb, gb, 4);    //Front
-     five_bit_ch += copy_bits(pb, gb, 4);    //Side
-     five_bit_ch += copy_bits(pb, gb, 4);    //Back
-     four_bit_ch  = copy_bits(pb, gb, 2);    //LFE
-     four_bit_ch += copy_bits(pb, gb, 3);    //Data
-     five_bit_ch += copy_bits(pb, gb, 4);    //Coupling
-     if (copy_bits(pb, gb, 1))               //Mono Mixdown
-         copy_bits(pb, gb, 4);
-     if (copy_bits(pb, gb, 1))               //Stereo Mixdown
-         copy_bits(pb, gb, 4);
-     if (copy_bits(pb, gb, 1))               //Matrix Mixdown
-         copy_bits(pb, gb, 3);
-     for (bits = five_bit_ch*5+four_bit_ch*4; bits > 16; bits -= 16)
-         copy_bits(pb, gb, 16);
-     if (bits)
-         copy_bits(pb, gb, bits);
-     avpriv_align_put_bits(pb);
-     align_get_bits(gb);
-     comment_size = copy_bits(pb, gb, 8);
-     for (; comment_size > 0; comment_size--)
-         copy_bits(pb, gb, 8);
-     return put_bits_count(pb) - offset;
- }
@@@ -112,9 -105,47 +115,47 @@@ enum AudioObjectType 
      AOT_USAC,                  ///< N                       Unified Speech and Audio Coding
  };
  
 -#define MAX_PCE_SIZE 304 ///<Maximum size of a PCE including the 3-bit ID_PCE
 +#define MAX_PCE_SIZE 320 ///<Maximum size of a PCE including the 3-bit ID_PCE
                           ///<marker and the comment
  
- int avpriv_copy_pce_data(PutBitContext *pb, GetBitContext *gb);
+ static av_always_inline unsigned int ff_pce_copy_bits(PutBitContext *pb,
+                                                       GetBitContext *gb,
+                                                       int bits)
+ {
+     unsigned int el = get_bits(gb, bits);
+     put_bits(pb, bits, el);
+     return el;
+ }
+ static inline int ff_copy_pce_data(PutBitContext *pb, GetBitContext *gb)
+ {
+     int five_bit_ch, four_bit_ch, comment_size, bits;
+     int offset = put_bits_count(pb);
+     ff_pce_copy_bits(pb, gb, 10);               // Tag, Object Type, Frequency
+     five_bit_ch  = ff_pce_copy_bits(pb, gb, 4); // Front
+     five_bit_ch += ff_pce_copy_bits(pb, gb, 4); // Side
+     five_bit_ch += ff_pce_copy_bits(pb, gb, 4); // Back
+     four_bit_ch  = ff_pce_copy_bits(pb, gb, 2); // LFE
+     four_bit_ch += ff_pce_copy_bits(pb, gb, 3); // Data
+     five_bit_ch += ff_pce_copy_bits(pb, gb, 4); // Coupling
+     if (ff_pce_copy_bits(pb, gb, 1))            // Mono Mixdown
+         ff_pce_copy_bits(pb, gb, 4);
+     if (ff_pce_copy_bits(pb, gb, 1))            // Stereo Mixdown
+         ff_pce_copy_bits(pb, gb, 4);
+     if (ff_pce_copy_bits(pb, gb, 1))            // Matrix Mixdown
+         ff_pce_copy_bits(pb, gb, 3);
+     for (bits = five_bit_ch*5+four_bit_ch*4; bits > 16; bits -= 16)
+         ff_pce_copy_bits(pb, gb, 16);
+     if (bits)
+         ff_pce_copy_bits(pb, gb, bits);
+     avpriv_align_put_bits(pb);
+     align_get_bits(gb);
+     comment_size = ff_pce_copy_bits(pb, gb, 8);
+     for (; comment_size > 0; comment_size--)
+         ff_pce_copy_bits(pb, gb, 8);
+     return put_bits_count(pb) - offset;
+ }
  
  #endif /* AVCODEC_MPEG4AUDIO_H */
Simple merge
@@@ -124,11 -111,7 +124,11 @@@ static void latm_write_frame_header(AVF
              avpriv_copy_bits(bs, par->extradata, ctx->off + 3);
  
              if (!ctx->channel_conf) {
-                 avpriv_copy_pce_data(bs, &gb);
 +                GetBitContext gb;
 +                int ret = init_get_bits8(&gb, par->extradata, par->extradata_size);
 +                av_assert0(ret >= 0); // extradata size has been checked already, so this should not fail
 +                skip_bits_long(&gb, ctx->off + 3);
+                 ff_copy_pce_data(bs, &gb);
              }
          }