[FFmpeg-devel] [PATCH] RV30/40 decoder

Kostya kostya.shishkov
Sat Nov 24 15:43:56 CET 2007


On Fri, Nov 23, 2007 at 09:46:19PM +0100, Michael Niedermayer wrote:
> On Sun, Nov 18, 2007 at 11:11:24AM +0200, Kostya wrote:
> > Well, it roughly the same feature-wise as it was,
> > I just don't think I will improve it soon, yet
> > it is playable (and maybe will attract samples
> > and patches, I'm an optimist).
> 
> rv30data.h
> ok, commit

So I can commit only header filer as they get approved, right?
 
[...] 
 
> rv34data.h
> > +/**
> > + * Chroma quantizer values
> > + * Second table is used for DC-only blocks
> > + */
> > +static const uint8_t rv34_chroma_quant[2][32] = {
> > + {  0,  1,  2,  3,  4,  5,  6,  7,  8,  9, 10, 11, 12, 13, 14, 15,
> > +   16, 17, 17, 18, 19, 20, 20, 21, 22, 22, 23, 23, 24, 24, 25, 25 },
> > + {  0,  0,  0,  1,  2,  3,  4,  5,  6,  7,  8,  9, 10, 11, 12, 13,
> > +   14, 15, 15, 16, 17, 18, 18, 19, 20, 20, 21, 21, 22, 22, 23, 23 }
> > +};
> 
> the name is inconsistant to rv30_luma_dc_quant
> or if its noz just for dc then the comment is wrong

Comment is wrong, first subtable is for scaling AC, the second one is for
scaling DC.
 
> except these rv34data.h can be commited
> 
> 
> rv40vlc2.h
> ok, commit
> 
> 
> rv34vlc.h
> > +// INTRA TABLES
> > +// BLOCK 0
> > +static const uint8_t rv34_table_0_0_0[CBPPAT_VLC_SIZE] =
> 
> comment is not doxygen compatible
> 
> also the use of names like rv34_table_X_Y_Z instead of tables with
> proper names is unacceptable
> 
> non table data not yet reviewed

Here's new version.
 
> [...]
> -- 
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
-------------- next part --------------
A non-text attachment was scrubbed...
Name: rv34vlc.h.bz2
Type: application/x-bzip2
Size: 20002 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071124/84c137ac/attachment.bin>



More information about the ffmpeg-devel mailing list