January 2009 - FFmpeg-soc - lists.ffmpeg.org

[soc]: r3970 - in wmapro: wma3data.h wma3dec.c
by faust3 24 Jan '09

24 Jan '09

Author: faust3 Date: Sat Jan 17 13:55:52 2009 New Revision: 3970 Log: moved scale factor run length level and run tables next to the huffman tables and renamed them to ff_wma3_scale_rl_level / run Modified: wmapro/wma3data.h wmapro/wma3dec.c Modified: wmapro/wma3data.h ============================================================================== --- wmapro/wma3data.h Sat Jan 17 13:46:01 2009 (r3969) +++ wmapro/wma3data.h Sat Jan 17 13:55:52 2009 (r3970) @@ -118,6 +118,27 @@ static const uint8_t ff_wma3_scale_rl_hu }; +static const uint8_t ff_wma3_scale_rl_run[] = { + 0, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, + 18, 19, 20, 21, 22, 23, 24, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, + 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 0, 1, 2, 3, 4, 5, + 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, + 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, + 19, 20, 21, 22, 23, 24, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 0, 1, + 0, 1, 0, 1, +}; + +static const uint8_t ff_wma3_scale_rl_level[] = { + 1, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, + 2, 2, 2, 2, 2, 2, 2, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, + 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 4, 4, 4, 4, 4, 4, + 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, + 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, + 5, 5, 5, 5, 5, 5, 6, 6, 6, 6, 6, 6, 6, 6, 6, 6, 6, 7, 7, + 8, 8, 9, 9, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, +}; + + #define FF_WMA3_COEF0_SIZE 244 #define FF_WMA3_COEF0_MAXBITS 22 static const uint32_t ff_wma3_coef0_huffcodes[FF_WMA3_COEF0_SIZE] = { @@ -521,27 +542,6 @@ static const uint8_t ff_wma3_level_1[] = }; -static const uint8_t ff_wma3_run_mask[] = { - 0, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, - 18, 19, 20, 21, 22, 23, 24, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, - 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 0, 1, 2, 3, 4, 5, - 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, - 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, - 19, 20, 21, 22, 23, 24, 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 0, 1, - 0, 1, 0, 1, -}; - -static const uint8_t ff_wma3_level_mask[] = { - 1, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, - 2, 2, 2, 2, 2, 2, 2, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, - 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 3, 4, 4, 4, 4, 4, 4, - 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, 4, - 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, 5, - 5, 5, 5, 5, 5, 5, 6, 6, 6, 6, 6, 6, 6, 6, 6, 6, 6, 7, 7, - 8, 8, 9, 9, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -}; - - static const float ff_wma3_default_decorrelation_matrices[] = { 1.000000, 0.707031, -0.707031, 0.707031, 0.707031, 0.578125, 0.707031, 0.410156, 0.578125, -0.707031, 0.410156, 0.578125, 0.000000, -0.816406, Modified: wmapro/wma3dec.c ============================================================================== --- wmapro/wma3dec.c Sat Jan 17 13:46:01 2009 (r3969) +++ wmapro/wma3dec.c Sat Jan 17 13:55:52 2009 (r3970) @@ -914,8 +914,8 @@ static int wma_decode_scale_factors(WMA3 }else if(idx == 1){ break; }else{ - skip = ff_wma3_run_mask[idx-2]; - level_mask = ff_wma3_level_mask[idx-2]; + skip = ff_wma3_scale_rl_run[idx-2]; + level_mask = ff_wma3_scale_rl_level[idx-2]; val = get_bits(gb,1)-1; }

3 3

[soc]: r3977 - in wmapro: wma3.h wma3dec.c
by faust3 24 Jan '09

24 Jan '09

Author: faust3 Date: Sat Jan 17 16:56:55 2009 New Revision: 3977 Log: removed channel mask from the context Modified: wmapro/wma3.h wmapro/wma3dec.c Modified: wmapro/wma3.h ============================================================================== --- wmapro/wma3.h Sat Jan 17 16:29:58 2009 (r3976) +++ wmapro/wma3.h Sat Jan 17 16:56:55 2009 (r3977) @@ -143,7 +143,6 @@ typedef struct WMA3DecodeContext { /** extradata */ unsigned int decode_flags; //< used compression features - unsigned int dwChannelMask; uint8_t sample_bit_depth; //< bits per sample /** general frame info */ Modified: wmapro/wma3dec.c ============================================================================== --- wmapro/wma3dec.c Sat Jan 17 16:29:58 2009 (r3976) +++ wmapro/wma3dec.c Sat Jan 17 16:56:55 2009 (r3977) @@ -37,7 +37,6 @@ static void dump_context(WMA3DecodeConte #define PRINT(a,b) av_log(s->avctx,AV_LOG_ERROR," %s = %d\n", a, b); #define PRINT_HEX(a,b) av_log(s->avctx,AV_LOG_ERROR," %s = %x\n", a, b); - PRINT_HEX("ed channelmask",s->dwChannelMask); PRINT("ed sample bit depth",s->sample_bit_depth); PRINT_HEX("ed decode flags",s->decode_flags); PRINT("samples per frame",s->samples_per_frame); @@ -133,6 +132,7 @@ static av_cold int wma3_decode_init(AVCo WMA3DecodeContext *s = avctx->priv_data; uint8_t *edata_ptr = avctx->extradata; int* sfb_offsets; + unsigned int channel_mask; int i; s->avctx = avctx; @@ -142,7 +142,7 @@ static av_cold int wma3_decode_init(AVCo s->sample_bit_depth = 16; //avctx->bits_per_sample; if (avctx->extradata_size >= 18) { s->decode_flags = AV_RL16(edata_ptr+14); - s->dwChannelMask = AV_RL32(edata_ptr+2); + channel_mask = AV_RL32(edata_ptr+2); // s->sample_bit_depth = AV_RL16(edata_ptr); /** dump the extradata */ @@ -195,10 +195,10 @@ static av_cold int wma3_decode_init(AVCo /** extract lfe channel position */ s->lfe_channel = -1; - if(s->dwChannelMask & 8){ + if(channel_mask & 8){ unsigned int mask = 1; for(i=0;i<32;i++){ - if(s->dwChannelMask & mask) + if(channel_mask & mask) ++s->lfe_channel; if(mask & 8) break;

3 2

[soc]: r3996 - dvbmuxer/mpegenc.c
by diego 23 Jan '09

23 Jan '09

Author: diego Date: Fri Jan 23 01:38:23 2009 New Revision: 3996 Log: #ifdef --> #if where appropriate to sync with build system changes Modified: dvbmuxer/mpegenc.c Modified: dvbmuxer/mpegenc.c ============================================================================== --- dvbmuxer/mpegenc.c Fri Jan 23 01:35:26 2009 (r3995) +++ dvbmuxer/mpegenc.c Fri Jan 23 01:38:23 2009 (r3996) @@ -860,7 +860,7 @@ static int mpeg_mux_end(AVFormatContext return 0; } -#ifdef CONFIG_MPEG1SYSTEM_MUXER +#if CONFIG_MPEG1SYSTEM_MUXER AVOutputFormat mpeg1system_muxer = { "mpeg", NULL_IF_CONFIG_SMALL("MPEG-1 System format"), @@ -874,7 +874,8 @@ AVOutputFormat mpeg1system_muxer = { mpeg_mux_end, }; #endif -#ifdef CONFIG_MPEG1VCD_MUXER + +#if CONFIG_MPEG1VCD_MUXER AVOutputFormat mpeg1vcd_muxer = { "vcd", NULL_IF_CONFIG_SMALL("MPEG-1 System format (VCD)"), @@ -888,7 +889,8 @@ AVOutputFormat mpeg1vcd_muxer = { mpeg_mux_end, }; #endif -#ifdef CONFIG_MPEG2VOB_MUXER + +#if CONFIG_MPEG2VOB_MUXER AVOutputFormat mpeg2vob_muxer = { "vob", NULL_IF_CONFIG_SMALL("MPEG-2 PS format (VOB)"), @@ -904,7 +906,7 @@ AVOutputFormat mpeg2vob_muxer = { #endif /* Same as mpeg2vob_mux except that the pack size is 2324 */ -#ifdef CONFIG_MPEG2SVCD_MUXER +#if CONFIG_MPEG2SVCD_MUXER AVOutputFormat mpeg2svcd_muxer = { "svcd", NULL_IF_CONFIG_SMALL("MPEG-2 PS format (VOB)"), @@ -920,7 +922,7 @@ AVOutputFormat mpeg2svcd_muxer = { #endif /* Same as mpeg2vob_mux except the 'is_dvd' flag is set to produce NAV pkts */ -#ifdef CONFIG_MPEG2DVD_MUXER +#if CONFIG_MPEG2DVD_MUXER AVOutputFormat mpeg2dvd_muxer = { "dvd", NULL_IF_CONFIG_SMALL("MPEG-2 PS format (DVD VOB)"),

1 0

[soc]: r3995 - dirac/libavcodec/diracenc.c
by diego 23 Jan '09

23 Jan '09

Author: diego Date: Fri Jan 23 01:35:26 2009 New Revision: 3995 Log: Remove useless #ifdef. Modified: dirac/libavcodec/diracenc.c Modified: dirac/libavcodec/diracenc.c ============================================================================== --- dirac/libavcodec/diracenc.c Fri Jan 23 01:12:17 2009 (r3994) +++ dirac/libavcodec/diracenc.c Fri Jan 23 01:35:26 2009 (r3995) @@ -953,7 +953,6 @@ static int encode_frame(AVCodecContext * return size; } -#ifdef CONFIG_ENCODERS AVCodec dirac_encoder = { "dirac", CODEC_TYPE_VIDEO, @@ -964,4 +963,3 @@ AVCodec dirac_encoder = { encode_end, .pix_fmts = (enum PixelFormat[]) {PIX_FMT_YUV420P, -1} }; -#endif

1 0

[PATCH] [soc: dvbmuxer] Comply to trunk r16600
by Tomer Barletz 23 Jan '09

23 Jan '09

In r16600 all ENABLE_* was renamed to CONFIG_* in file mpegenc.c. Apply to soc branch too. Tomer

2 1

[soc]: r3994 - dvbmuxer/mpegenc.c
by diego 23 Jan '09

23 Jan '09

Author: diego Date: Fri Jan 23 01:12:17 2009 New Revision: 3994 Log: Rename ENABLE preprocessor directives to CONFIG_. patch by Tomer Barletz, barletz gmail com Modified: dvbmuxer/mpegenc.c Modified: dvbmuxer/mpegenc.c ============================================================================== --- dvbmuxer/mpegenc.c Tue Jan 20 22:14:32 2009 (r3993) +++ dvbmuxer/mpegenc.c Fri Jan 23 01:12:17 2009 (r3994) @@ -273,12 +273,12 @@ static int mpeg_mux_init(AVFormatContext int video_bitrate; s->packet_number = 0; - s->is_vcd = (ENABLE_MPEG1VCD_MUXER && ctx->oformat == &mpeg1vcd_muxer); - s->is_svcd = (ENABLE_MPEG2SVCD_MUXER && ctx->oformat == &mpeg2svcd_muxer); - s->is_mpeg2 = ((ENABLE_MPEG2VOB_MUXER && ctx->oformat == &mpeg2vob_muxer) || - (ENABLE_MPEG2DVD_MUXER && ctx->oformat == &mpeg2dvd_muxer) || - (ENABLE_MPEG2SVCD_MUXER && ctx->oformat == &mpeg2svcd_muxer)); - s->is_dvd = (ENABLE_MPEG2DVD_MUXER && ctx->oformat == &mpeg2dvd_muxer); + s->is_vcd = (CONFIG_MPEG1VCD_MUXER && ctx->oformat == &mpeg1vcd_muxer); + s->is_svcd = (CONFIG_MPEG2SVCD_MUXER && ctx->oformat == &mpeg2svcd_muxer); + s->is_mpeg2 = ((CONFIG_MPEG2VOB_MUXER && ctx->oformat == &mpeg2vob_muxer) || + (CONFIG_MPEG2DVD_MUXER && ctx->oformat == &mpeg2dvd_muxer) || + (CONFIG_MPEG2SVCD_MUXER && ctx->oformat == &mpeg2svcd_muxer)); + s->is_dvd = (CONFIG_MPEG2DVD_MUXER && ctx->oformat == &mpeg2dvd_muxer); if(ctx->packet_size) s->packet_size = ctx->packet_size;

1 0

[soc]: r3993 - libavfilter/vf_format.c
by stefano 20 Jan '09

20 Jan '09

Author: stefano Date: Tue Jan 20 22:14:32 2009 New Revision: 3993 Log: Fix the check for an unknown pixel format and the resulting buffer underflow/crash. Modified: libavfilter/vf_format.c Modified: libavfilter/vf_format.c ============================================================================== --- libavfilter/vf_format.c Mon Jan 19 19:08:16 2009 (r3992) +++ libavfilter/vf_format.c Tue Jan 20 22:14:32 2009 (r3993) @@ -53,7 +53,7 @@ static int init(AVFilterContext *ctx, co sep ++; } - if(fmt >= PIX_FMT_NB) { + if(fmt == PIX_FMT_NONE) { av_log(ctx, AV_LOG_ERROR, "unknown pixel format\n"); continue; }

1 0

[soc] [propchange]: r3970 - svn:log
by diego 20 Jan '09

20 Jan '09

Author: diego Revision: 3970 Property Name: svn:log Action: modified Property diff: --- old property value +++ new property value @@ -1 +1,2 @@ -moved scale factor run length level and run tables next to the huffman tables and renamed them to ff_wma3_scale_rl_level / run \ No newline at end of file +Moved scale factor run length level and run tables next to the huffman +tables and renamed them to ff_wma3_scale_rl_level / run.

1 0

[soc]: r3992 - in wmapro: wma3.h wma3data.h wma3dec.c
by diego 19 Jan '09

19 Jan '09

Author: diego Date: Mon Jan 19 19:08:16 2009 New Revision: 3992 Log: spelling/grammar/whitespace cosmetics Modified: wmapro/wma3.h wmapro/wma3data.h wmapro/wma3dec.c Modified: wmapro/wma3.h ============================================================================== --- wmapro/wma3.h Mon Jan 19 17:43:27 2009 (r3991) +++ wmapro/wma3.h Mon Jan 19 19:08:16 2009 (r3992) @@ -30,7 +30,7 @@ #define MAX_SUBFRAMES 32 //< max number of subframes per channel #define MAX_BANDS 29 //< max number of scale factor bands -/* size of block defines taken from wma.h*/ +/* size of block defines taken from wma.h */ #define BLOCK_MIN_BITS 7 #define BLOCK_MAX_BITS 12 #define BLOCK_MAX_SIZE (1 << BLOCK_MAX_BITS) @@ -43,10 +43,10 @@ typedef struct { int prev_block_len; //< length of the previous block uint8_t transmit_coefs; //< transmit coefficients uint8_t num_subframes; //< number of subframes - uint16_t subframe_len[MAX_SUBFRAMES]; //< subframe len in samples + uint16_t subframe_len[MAX_SUBFRAMES]; //< subframe length in samples uint16_t subframe_offset[MAX_SUBFRAMES]; //< subframe position uint8_t cur_subframe; //< subframe index - uint16_t channel_len; //< channel len in samples + uint16_t channel_len; //< channel length in samples uint16_t decoded_samples; //< already processed samples uint8_t grouped; //< channel is part of a group int quant_step_modifier; @@ -56,7 +56,7 @@ typedef struct { int max_scale_factor; //< maximum scale factor int scale_factors[MAX_BANDS]; //< scale factor values int resampled_scale_factors[MAX_BANDS]; //< scale factors from a previous block - int scale_factor_block_len; //< sf reference block len + int scale_factor_block_len; //< sf reference block length DECLARE_ALIGNED_16(float, coeffs[BLOCK_MAX_SIZE]); //< decode buffer DECLARE_ALIGNED_16(float, out[2*BLOCK_MAX_SIZE]); //< output buffer } WMA3ChannelCtx; @@ -66,11 +66,11 @@ typedef struct { */ typedef struct { int nb_channels; - int no_rotation; //< controls the type of the transform - int transform; //< also controls the type of the transform + int no_rotation; //< controls the type of the transform + int transform; //< also controls the type of the transform char transform_band[MAX_BANDS]; //< controls if the transform is enabled for a certain band char rotation_offset[MAX_CHANNELS * MAX_CHANNELS]; - char positive[MAX_CHANNELS * MAX_CHANNELS]; //< fixme for what are these numbers used? + char positive[MAX_CHANNELS * MAX_CHANNELS]; //< FIXME: What are these numbers used for? float decorrelation_matrix[MAX_CHANNELS*MAX_CHANNELS]; char use_channel[MAX_CHANNELS]; } WMA3ChannelGroup; @@ -82,23 +82,23 @@ typedef struct WMA3DecodeContext { /** generic decoder variables */ AVCodecContext* avctx; //< codec context for av_log DSPContext dsp; - MDCTContext mdct_ctx[BLOCK_NB_SIZES]; //< mdct context per block size + MDCTContext mdct_ctx[BLOCK_NB_SIZES]; //< MDCT context per block size float* windows[BLOCK_NB_SIZES]; //< window per block size - VLC sf_vlc; //< scale factor dpcm vlc + VLC sf_vlc; //< scale factor DPCM vlc VLC sf_rl_vlc; //< scale factor run length vlc - VLC vec4_vlc; //< 4 coefs per symbol - VLC vec2_vlc; //< 2 coefs per symbol - VLC vec1_vlc; //< 1 coef per symbol - VLC coef_vlc[2]; //< coef run length vlc codes + VLC vec4_vlc; //< 4 coefficients per symbol + VLC vec2_vlc; //< 2 coefficients per symbol + VLC vec1_vlc; //< 1 coefficient per symbol + VLC coef_vlc[2]; //< coefficient run length vlc codes int coef_max[2]; //< max length of vlc codes - /** frame size dependant frame information (set during init) */ + /** frame size dependent frame information (set during initialization) */ uint8_t lossless; //< lossless mode unsigned int decode_flags; //< used compression features - uint8_t len_prefix; //< frame is prefixed with its len - uint8_t dynamic_range_compression;//< frame contains drc data + uint8_t len_prefix; //< frame is prefixed with its length + uint8_t dynamic_range_compression;//< frame contains DRC data uint8_t sample_bit_depth; //< bits per sample - uint16_t samples_per_frame; //< number of outputed samples + uint16_t samples_per_frame; //< number of samples to output uint16_t log2_frame_size; //< frame size int8_t nb_channels; //< number of channels int8_t lfe_channel; //< lfe channel index @@ -128,7 +128,7 @@ typedef struct WMA3DecodeContext { int buf_bit_size; //< buffer size in bits int16_t* samples; //< current samplebuffer pointer int16_t* samples_end; //< maximum samplebuffer pointer - uint8_t drc_gain; //< gain for the drc tool + uint8_t drc_gain; //< gain for the DRC tool uint8_t no_tiling; //< frames contain subframes int skip_frame; //< skip output step int parsed_all_subframes; //< all subframes decoded? Modified: wmapro/wma3data.h ============================================================================== --- wmapro/wma3data.h Mon Jan 19 17:43:27 2009 (r3991) +++ wmapro/wma3data.h Mon Jan 19 19:08:16 2009 (r3992) @@ -40,7 +40,7 @@ static const uint16_t ff_wma3_critical_f /** - * @name huffman tables for dpcm coded scale factors + * @name huffman tables for DPCM-coded scale factors * @{ */ #define FF_WMA3_HUFF_SCALE_SIZE 121 @@ -410,7 +410,7 @@ static const uint8_t ff_wma3_coef1_level /** - * @name huffman and vector lookup tables for vector coded coefficients + * @name huffman and vector lookup tables for vector-coded coefficients * @{ */ #define FF_WMA3_HUFF_VEC4_SIZE 127 Modified: wmapro/wma3dec.c ============================================================================== --- wmapro/wma3dec.c Mon Jan 19 17:43:27 2009 (r3991) +++ wmapro/wma3dec.c Mon Jan 19 19:08:16 2009 (r3992) @@ -91,7 +91,7 @@ static int get_samples_per_frame(int sam } /** - *@brief Uninitialize the decoder and free all ressources. + *@brief Uninitialize the decoder and free all resources. *@param avctx codec context *@return 0 on success, < 0 otherwise */ @@ -142,8 +142,8 @@ static av_cold int wma3_decode_init(AVCo s->avctx = avctx; dsputil_init(&s->dsp, avctx); - /* FIXME is this really the right thing todo for 24 bit? */ - s->sample_bit_depth = 16; //avctx->bits_per_sample; + /* FIXME: is this really the right thing to do for 24 bits? */ + s->sample_bit_depth = 16; // avctx->bits_per_sample; if (avctx->extradata_size >= 18) { s->decode_flags = AV_RL16(edata_ptr+14); channel_mask = AV_RL32(edata_ptr+2); @@ -161,7 +161,7 @@ static av_cold int wma3_decode_init(AVCo } /** generic init */ - s->packet_loss = 0; //FIXME unneeded + s->packet_loss = 0; // FIXME: unneeded s->log2_frame_size = av_log2(avctx->block_align*8)+1; /** frame info */ @@ -169,7 +169,7 @@ static av_cold int wma3_decode_init(AVCo s->len_prefix = s->decode_flags & 0x40; if(!s->len_prefix){ - av_log(avctx, AV_LOG_ERROR, "file has no len prefix please report\n"); + av_log(avctx, AV_LOG_ERROR, "file has no length prefix, please report\n"); return -1; } @@ -285,9 +285,9 @@ static av_cold int wma3_decode_init(AVCo } - /** scale factors can be shared between blocks of different size - as every block has a different scale factor band layout - the matrix sf_offsets is needed to find the correct scale factor + /** Scale factors can be shared between blocks of different size + as every block has a different scale factor band layout. + The matrix sf_offsets is needed to find the correct scale factor. */ for(i=0;i<s->num_possible_block_sizes;i++){ @@ -306,11 +306,11 @@ static av_cold int wma3_decode_init(AVCo } } - /** init MDCT, FIXME only init needed sizes */ + /** init MDCT, FIXME: only init needed sizes */ for(i = 0; i < BLOCK_NB_SIZES; i++) ff_mdct_init(&s->mdct_ctx[i], BLOCK_MIN_BITS+1+i, 1); - /** init MDCT windows : simple sinus window */ + /** init MDCT windows: simple sinus window */ for(i=0 ; i<BLOCK_NB_SIZES ; i++) { int n; n = 1 << (BLOCK_MAX_BITS - i); @@ -332,7 +332,7 @@ static av_cold int wma3_decode_init(AVCo s->subwoofer_cutoff = s->subwoofer_cutoffs[0]; - /** set up decorrelation matrices */ + /** set up decorrelation matrixes */ s->def_decorrelation_mat = av_mallocz(sizeof(int) * (s->nb_channels + 1)); if(!s->def_decorrelation_mat){ av_log(avctx, AV_LOG_ERROR, "failed to allocate decorrelation matrix\n"); @@ -345,7 +345,7 @@ static av_cold int wma3_decode_init(AVCo const float* tab = ff_wma3_default_decorrelation_matrices; s->def_decorrelation_mat[i] = av_mallocz(sizeof(float) * i); if(!s->def_decorrelation_mat[i]){ - av_log(avctx, AV_LOG_ERROR, "failed to setup decorrelation mat\n"); + av_log(avctx, AV_LOG_ERROR, "failed to set up decorrelation matrix\n"); wma3_decode_end(avctx); return -1; } @@ -391,26 +391,25 @@ static av_cold int wma3_decode_init(AVCo } /** - *@brief Decode how the data in the frame is split into subframes - * every WMA frame contains the encoded data for a fixed number of - * samples per channel the data for every channel might be split - * into several subframes this function will reconstruct the list of - * subframes for every channel + *@brief Decode how the data in the frame is split into subframes. + * Every WMA frame contains the encoded data for a fixed number of + * samples per channel. The data for every channel might be split + * into several subframes. This function will reconstruct the list of + * subframes for every channel. * - * If the subframes are not evenly split the algorithm estimates the + * If the subframes are not evenly split, the algorithm estimates the * channels with the lowest number of total samples. - * Afterwards for each of these channels a bit is read from the + * Afterwards, for each of these channels a bit is read from the * bitstream that indicates if the channel contains a frame with the * next subframe size that is going to be read from the bitstream or not. - * If a channel contains such a subframe the subframe size gets added to + * If a channel contains such a subframe, the subframe size gets added to * the channel's subframe list. * The algorithm repeats these steps until the frame is properly divided * between the individual channels. * - * *@param s context *@param gb current get bit context - *@return 0 on success < 0 in case of an error + *@return 0 on success, < 0 in case of an error */ static int wma_decode_tilehdr(WMA3DecodeContext *s, GetBitContext* gb) { @@ -423,16 +422,16 @@ static int wma_decode_tilehdr(WMA3Decode s->channel[c].channel_len = 0; } - /** handle the easy case whith one constant sized subframe per channel */ + /** handle the easy case with one constant-sized subframe per channel */ if(s->max_num_subframes == 1){ for(c=0;c<s->nb_channels;c++){ s->channel[c].num_subframes = 1; s->channel[c].subframe_len[0] = s->samples_per_frame; s->channel[c].channel_len = 0; } - }else{ /** subframe len and number of subframes is not constant */ - int subframe_len_bits = 0; /** bits needed for the subframe len */ - int subframe_len_zero_bit = 0; /** how many of the len bits indicate + }else{ /** subframe length and number of subframes is not constant */ + int subframe_len_bits = 0; /** bits needed for the subframe length */ + int subframe_len_zero_bit = 0; /** how many of the length bits indicate if the subframe is zero */ /** calculate subframe len bits */ @@ -452,7 +451,7 @@ static int wma_decode_tilehdr(WMA3Decode int num_channels = 0; int subframe_len = s->samples_per_frame / s->max_num_subframes; - /** find channel with the smallest overall len */ + /** find channel with the smallest overall length */ for(c=0;c<s->nb_channels;c++){ if(min_channel_len > s->channel[c].channel_len) min_channel_len = s->channel[c].channel_len; @@ -467,7 +466,7 @@ static int wma_decode_tilehdr(WMA3Decode num_subframes_per_channel = 1; num_channels = s->nb_channels; }else{ - /** count how many channels have the minimum len */ + /** count how many channels have the minimum length */ for(c=0;c<s->nb_channels;c++){ if(min_channel_len == s->channel[c].channel_len){ ++num_channels; @@ -486,9 +485,9 @@ static int wma_decode_tilehdr(WMA3Decode if(num_subframes_per_channel != 1){ int total_num_bits = num_channels; tileinfo = 0; - /** for every channel with the minimum len 1 bit is - transmitted that informs us if the channel - contains a subframe with the next subframe_len */ + /** For every channel with the minimum length, 1 bit + is transmitted that informs us if the channel + contains a subframe with the next subframe_len. */ while(total_num_bits){ int num_bits = total_num_bits; if(num_bits > 32) @@ -500,11 +499,11 @@ static int wma_decode_tilehdr(WMA3Decode } } - /** if the frames are not evenly split get the next subframe len - from the bitstream */ + /** if the frames are not evenly split, get the next subframe + length from the bitstream */ if(subframe_len != missing_samples / num_channels){ int log2_subframe_len = 0; - /* 1 bit indicates if the subframe len is zero */ + /* 1 bit indicates if the subframe length is zero */ if(subframe_len_zero_bit){ if(get_bits1(gb)){ log2_subframe_len = get_bits(gb,subframe_len_bits-1); @@ -522,7 +521,7 @@ static int wma_decode_tilehdr(WMA3Decode s->samples_per_frame / (1 << log2_subframe_len); } - /** sanity check the len */ + /** sanity check the length */ if(subframe_len < s->min_samples_per_subframe || subframe_len > s->samples_per_frame){ av_log(s->avctx, AV_LOG_ERROR, @@ -631,7 +630,9 @@ static int wma_decode_channel_transform( /** done decode channel mask */ - /* decide x form type FIXME port this to float, all rotations should lie on the unit circle*/ + /* decide x form type + FIXME: port this to float, all rotations should lie + on the unit circle */ if(chgroup->nb_channels == 1){ chgroup->no_rotation = 1; chgroup->transform = 2; @@ -650,7 +651,7 @@ static int wma_decode_channel_transform( }else{ chgroup->no_rotation = 1; chgroup->transform = 1; - chgroup->decorrelation_matrix[0] = 0.70703125; //FIXME cos(pi/4) + chgroup->decorrelation_matrix[0] = 0.70703125; // FIXME: cos(pi/4) chgroup->decorrelation_matrix[1] = -0.70703125; chgroup->decorrelation_matrix[2] = 0.70703125; chgroup->decorrelation_matrix[3] = 0.70703125; @@ -790,7 +791,7 @@ static int decode_coeffs(WMA3DecodeConte const uint8_t* run; const uint8_t* level; - av_log(s->avctx,AV_LOG_DEBUG,"decode coefs for channel %i\n",c); + av_log(s->avctx,AV_LOG_DEBUG,"decode coefficients for channel %i\n",c); s->getbit = gb; s->esc_len = av_log2(s->subframe_len -1) +1; @@ -855,7 +856,7 @@ static int decode_coeffs(WMA3DecodeConte } if( cur_coeff >= s->subframe_len ) // handled entire subframe -> quit return 0; - if ( last_write > s->subframe_len / 256 ) //switch to RL mode + if ( last_write > s->subframe_len / 256 ) // switch to RL mode rl_mode = 1; ++i; } @@ -900,13 +901,13 @@ static int wma_decode_scale_factors(WMA3 int i; s->channel[c].scale_factor_step = get_bits(gb,2) + 1; for(i=0;i<s->cValidBarkBand;i++){ - int val = get_vlc2(gb, s->sf_vlc.table, SCALEVLCBITS, ((FF_WMA3_HUFF_SCALE_MAXBITS+SCALEVLCBITS-1)/SCALEVLCBITS)); //dpcm coded + int val = get_vlc2(gb, s->sf_vlc.table, SCALEVLCBITS, ((FF_WMA3_HUFF_SCALE_MAXBITS+SCALEVLCBITS-1)/SCALEVLCBITS)); // DPCM-coded if(!i) s->channel[c].scale_factors[i] = 45 / s->channel[c].scale_factor_step + val - 60; else s->channel[c].scale_factors[i] = s->channel[c].scale_factors[i-1] + val - 60; } - }else{ // rl coded + }else{ // rl-coded int i; memcpy(s->channel[c].scale_factors,s->channel[c].resampled_scale_factors, 4 * s->cValidBarkBand); @@ -975,7 +976,7 @@ static void wma_calc_decorrelation_matri float v1 = tmp1[y]; float v2 = tmp2[y]; int n = chgroup->rotation_offset[offset + x]; - float cosv = sin(n*M_PI / 64.0); //FIXME use one table for this + float cosv = sin(n*M_PI / 64.0); // FIXME: use one table for this float sinv = -cos(n*M_PI / 64.0); chgroup->decorrelation_matrix[y + x * chgroup->nb_channels] = (v1 * cosv) + (v2 * sinv); @@ -1006,7 +1007,7 @@ static void wma_inverse_channel_transfor int b; for(b = 0; b < s->cValidBarkBand;b++){ int y; - if(s->chgroup[i].transform_band[b] == 1){ //M/S stereo + if(s->chgroup[i].transform_band[b] == 1){ // M/S stereo for(y=s->rgiBarkIndex[b];y<FFMIN(s->rgiBarkIndex[b+1], s->subframe_len);y++){ float v1 = s->channel[0].coeffs[y]; float v2 = s->channel[1].coeffs[y]; @@ -1042,7 +1043,7 @@ static void wma_inverse_channel_transfor for(b = 0; b < s->cValidBarkBand;b++){ int y; if(s->chgroup[i].transform_band[b] == 1){ - //multiply values with decorrelation_matrix + // multiply values with decorrelation_matrix for(y=s->rgiBarkIndex[b];y<FFMIN(s->rgiBarkIndex[b+1], s->subframe_len);y++){ float* matrix = s->chgroup[i].decorrelation_matrix; int m; @@ -1074,7 +1075,7 @@ static void wma_window(WMA3DecodeContext int c = s->channel_indices_for_cur_subframe[i]; int j = s->channel[c].cur_subframe; int x; - //FIXME use dsp.vector_fmul_window + // FIXME: use dsp.vector_fmul_window float* start; float* end; float* window; @@ -1098,7 +1099,7 @@ static void wma_window(WMA3DecodeContext // float* rs = &s->channel[c].out[s->samples_per_frame/2 + s->channel[c].subframe_offset[i]]; // printf("Dstart %i %i end %i win %i prev %i\n",s->frame_num+1, start - rs,end -rs,winlen,prev_block_len); - //FIXME untagle the windowing so dsp functions can be used + // FIXME: untangle the windowing so DSP functions can be used window = s->windows[av_log2(winlen)-BLOCK_MIN_BITS]; for(x=0;x<len;x++){ float ts = *start; @@ -1224,7 +1225,7 @@ static int wma_decode_subframe(WMA3Decod int quant; int sign = 1; int large_quant = 0; - if((get_bits(gb,1))){ /** FIXME might influence how often getvec4 may be called */ + if((get_bits(gb,1))){ /** FIXME: might influence how often getvec4 may be called */ av_log(s->avctx,AV_LOG_ERROR,"unsupported quant step coding\n"); return 0; } @@ -1317,9 +1318,9 @@ static int wma_decode_subframe(WMA3Decod } dst = &s->channel[c].out[s->samples_per_frame/2 + s->channel[c].subframe_offset[s->channel[c].cur_subframe]]; - ff_imdct_half(&s->mdct_ctx[av_log2(subframe_len)-BLOCK_MIN_BITS], dst, s->channel[c].coeffs); //DCTIV with reverse + ff_imdct_half(&s->mdct_ctx[av_log2(subframe_len)-BLOCK_MIN_BITS], dst, s->channel[c].coeffs); // DCTIV with reverse for(b=0;b<subframe_len;b++) - dst[b] /= subframe_len / 2; //FIXME try to remove this scaling + dst[b] /= subframe_len / 2; // FIXME: try to remove this scaling } } @@ -1366,7 +1367,7 @@ static int wma_decode_frame(WMA3DecodeCo if(s->len_prefix) len = get_bits(gb,s->log2_frame_size); - av_log(s->avctx,AV_LOG_DEBUG,"decoding frame with len %x\n",len); + av_log(s->avctx,AV_LOG_DEBUG,"decoding frame with length %x\n",len); /** decode tile information */ if(wma_decode_tilehdr(s,gb)){ @@ -1428,7 +1429,7 @@ static int wma_decode_frame(WMA3DecodeCo for(i = 0; i < s->nb_channels; i++) { int16_t* ptr; int incr = s->nb_channels; - /* FIXME what about other channel layouts? */ + /* FIXME: what about other channel layouts? */ const char layout[] = {0,1,4,5,2,3}; int chpos; float* iptr = s->channel[i].out; @@ -1446,7 +1447,7 @@ static int wma_decode_frame(WMA3DecodeCo ptr += incr; } - /** reuse second half of the imdct output for the next frame */ + /** reuse second half of the IMDCT output for the next frame */ memmove(&s->channel[i].out[0], &s->channel[i].out[s->samples_per_frame], s->samples_per_frame * sizeof(float)); } @@ -1457,7 +1458,7 @@ static int wma_decode_frame(WMA3DecodeCo s->samples += s->nb_channels * s->samples_per_frame; - //FIXME remove + // FIXME: remove av_log(s->avctx,AV_LOG_DEBUG,"frame[%i] skipping %i bits\n",s->frame_num,len - (get_bits_count(gb) - gb_start_count) - 1); if(len - (get_bits_count(gb) - gb_start_count) - 1 != 1) assert(0); @@ -1550,7 +1551,7 @@ static int wma3_decode_packet(AVCodecCon if(buf_size < avctx->block_align) return 0; - /** Parse packet header */ + /** parse packet header */ init_get_bits(&s->gb, buf, s->buf_bit_size); s->packet_sequence_number = get_bits(&s->gb, 4); s->bit5 = get_bits1(&s->gb); @@ -1569,7 +1570,7 @@ static int wma3_decode_packet(AVCodecCon } if (num_bits_prev_frame > 0) { - /** append the prev frame data to the remaining data from the + /** append the previous frame data to the remaining data from the previous packet to create a full frame */ save_bits(s,num_bits_prev_frame); av_log(avctx, AV_LOG_DEBUG, "accumulated %x bits of frame data\n", @@ -1587,7 +1588,7 @@ static int wma3_decode_packet(AVCodecCon s->prev_packet_bit_size); } - /** reset prev frame buffer */ + /** reset previous frame buffer */ s->prev_packet_bit_size = 0; s->packet_loss = 0; /** decode the rest of the packet */

1 0

[soc]: r3991 - nellyenc/nellymoserenc.c
by diego 19 Jan '09

19 Jan '09

Author: diego Date: Mon Jan 19 17:43:27 2009 New Revision: 3991 Log: samples.mplayerhq.hu --> samples.ffmpeg.org Modified: nellyenc/nellymoserenc.c Modified: nellyenc/nellymoserenc.c ============================================================================== --- nellyenc/nellymoserenc.c Mon Jan 19 16:55:47 2009 (r3990) +++ nellyenc/nellymoserenc.c Mon Jan 19 17:43:27 2009 (r3991) @@ -27,7 +27,7 @@ * by Bartlomiej Wolowiec * * Generic codec information: libavcodec/nellymoserdec.c - * Log search algorithm idea: http://samples.mplayerhq.hu/A-codecs/Nelly_Moser/ASAO/ASAO.zip + * Log search algorithm idea: http://samples.ffmpeg.org/A-codecs/Nelly_Moser/ASAO/ASAO.zip * (Copyright Joseph Artsimovich and UAB "DKD") * * for more information about nellymoser format, visit:

1 0