[FFmpeg-devel] [PATCH] TwinVQ decoder
Vitor Sessak
vitor1001
Mon May 18 18:32:55 CEST 2009
Michael Niedermayer wrote:
> On Wed, May 06, 2009 at 05:09:50PM +0200, Vitor Sessak wrote:
>> Vitor Sessak wrote:
>>> Hi,
>>> I'm running out of ideas on how to improve this code, so I'm submitting it
>>> for review. Among the things I'm not 100% happy with is the windowing mess
>>> and the set_interleave_table_* code duplication. Any suggestion is
>>> welcome.
>>> About using dsputils->vector_fmul, it gets two parameters (one for
>>> input/output and other for the window), and I need one that gets three...
>>> Finally, it would be very practical to start the review by the tables
>>> since they are so big that every time I send this patch the message need
>>> moderator approval...
>> Ping?
>>
>> New version attached...
> [...]
>> /**
>> * Parameters and tables that are different for each frame type
>> */
>> typedef struct {
>> uint8_t sub; ///< Number subblocks in each frame
>> const uint16_t *crb_tbl;
>> uint8_t n_crb;
>> const float *fw_cb;
>> uint8_t fw_n_div;
>> uint8_t fw_n_bit;
>> const float *cb0;
>> const float *cb1;
>> uint8_t cb_len_read;
>> } FrameMode;
>
> All the fields should have comments or better names
> this applies to more than this struct ...
Should be better now. Also taken in consideration Diego's comments...
-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: twinvq.c
Type: text/x-csrc
Size: 34396 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090518/8a02189b/attachment.c>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: twinvq_data.h
Type: text/x-chdr
Size: 12740 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090518/8a02189b/attachment.h>
More information about the ffmpeg-devel
mailing list