This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "AMR-WB decoder".
The branch, master has been updated via 5235ba66ad3c722abe03ed4659a84c54b4e644e3 (commit) from 2983a53df354f9bfbb8234805b23bd16b40c693a (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit 5235ba66ad3c722abe03ed4659a84c54b4e644e3 Author: Marcelo Povoa <marspeoples...@gmail.com> Date: Wed Jul 7 21:07:13 2010 -0300 Mark all temporary comments/notes with tag "XXX: " diff --git a/libavcodec/amrwbdata.h b/libavcodec/amrwbdata.h index 8422313..632589a 100644 --- a/libavcodec/amrwbdata.h +++ b/libavcodec/amrwbdata.h @@ -25,7 +25,7 @@ #include <stdint.h> #define LP_ORDER 16 ///< linear predictive coding filter order -#define MIN_ISF_SPACING 50.0 /* Taken from fixed-point 26.173, not sure */ +#define MIN_ISF_SPACING 50.0 /* XXX: Taken from fixed-point 26.173, not sure */ #define PRED_FACTOR (1.0/3.0) #define AMRWB_SUBFRAME_SIZE 64 ///< samples per subframe diff --git a/libavcodec/amrwbdec.c b/libavcodec/amrwbdec.c index a93fa9d..932ab9b 100644 --- a/libavcodec/amrwbdec.c +++ b/libavcodec/amrwbdec.c @@ -98,7 +98,7 @@ static enum Mode unpack_bitstream(AMRWBContext *ctx, const uint8_t *buf, /* AMR-WB Auxiliary Information */ ctx->fr_mode_ind = get_bits(&gb, 4); ctx->fr_mode_req = get_bits(&gb, 4); - ///Need to check conformity in mode_ind/mode_req and crc? + //XXX: Need to check conformity in mode_ind/mode_req and crc? ctx->fr_crc = get_bits(&gb, 8); data = (uint16_t *) &ctx->frame; @@ -257,7 +257,7 @@ static void isf_set_min_dist(float *isf, float min_spacing, int size) { static void interpolate_isp(double isp_q[4][LP_ORDER], double *isp4_past) { int i; - /* Did not used ff_weighted_vector_sumf because using double */ + /* XXX: Did not used ff_weighted_vector_sumf because using double */ for (i = 0; i < LP_ORDER; i++) isp_q[0][i] = 0.55 * isp4_past[i] + 0.45 * isp_q[3][i]; @@ -281,7 +281,7 @@ static void isp2lp(double isp[LP_ORDER], float *lp, int lp_half_order) { double pa[MAX_LP_HALF_ORDER+1], qa[MAX_LP_HALF_ORDER+1]; float *lp2 = lp + (lp_half_order << 1); double last_isp = isp[2 * lp_half_order - 1]; - double qa_old = 0; /* qa[i-2] assuming qa[-1] = 0, not mentioned in document */ + double qa_old = 0; /* XXX: qa[i-2] assuming qa[-1] = 0, not mentioned in document */ int i; ff_lsp2polyf(isp, pa, lp_half_order); @@ -332,7 +332,7 @@ static void decode_pitch_lag_high(int *lag_int, int *lag_frac, int pitch_index, *lag_int = (pitch_index + 1) >> 2; *lag_frac = pitch_index - (*lag_int << 2); *lag_int += *base_lag_int - 8; - /* Doesn't seem to need bounding according to TS 26.190 */ + /* XXX: Doesn't seem to need bounding according to TS 26.190 */ } } @@ -432,7 +432,7 @@ static void decode_pitch_vector(AMRWBContext *ctx, * @param code [in] Pulse index (no. of bits varies, see below) * @param m [in] (log2) Number of potential positions */ -//XXX : Some of these functions are simple and recurrent (used inline) +//XXX: Some of these functions are simple and recurrent (used inline) static inline void decode_1p_track(int *out, int code, int m) ///code: m+1 bits { @@ -471,7 +471,7 @@ static void decode_3p_track(int *out, int code, int m) ///code: 3m+1 bits * @param pulse_lo LSBs part of the pulse index array * @param mode mode of the current frame */ -// For now, uses the same AMRFixed struct from AMR-NB but +// XXX: For now, uses the same AMRFixed struct from AMR-NB but // the maximum number of pulses in it was increased to 24 static void decode_fixed_sparse(AMRFixed *fixed_sparse, const uint16_t *pulse_hi, const uint16_t *pulse_lo, const enum Mode mode) @@ -556,8 +556,9 @@ static int amrwb_decode_frame(AVCodecContext *avctx, void *data, int *data_size, for (sub = 0; sub < 4; sub++) { const AMRWBSubFrame *cur_subframe = &cf->subframe[sub]; + /* Decode adaptive codebook */ decode_pitch_vector(ctx, cur_subframe, sub); - + /* Decode innovative codebook */ decode_fixed_sparse(&fixed_sparse, cur_subframe->pul_ih, cur_subframe->pul_il, ctx->fr_cur_mode); } ----------------------------------------------------------------------- Summary of changes: libavcodec/amrwbdata.h | 2 +- libavcodec/amrwbdec.c | 15 ++++++++------- 2 files changed, 9 insertions(+), 8 deletions(-) hooks/post-receive -- AMR-WB decoder _______________________________________________ FFmpeg-soc mailing list FFmpeg-soc@mplayerhq.hu https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc