[FFmpeg-devel] [PATCH] H.264 data tables cleanup

Michael Niedermayer michaelni
Mon Nov 24 16:29:30 CET 2008


On Mon, Nov 24, 2008 at 01:24:50PM +0100, Diego Biurrun wrote:
> On Thu, Nov 20, 2008 at 10:10:11AM +0100, Michael Niedermayer wrote:
> > On Thu, Nov 20, 2008 at 09:46:29AM +0100, Diego Biurrun wrote:
> > > On Thu, Nov 20, 2008 at 09:08:17AM +0100, Michael Niedermayer wrote:
> > > > On Thu, Nov 20, 2008 at 12:49:03AM +0100, Diego Biurrun wrote:
> > > > > So I've (re)started working on splitting off svq3.c from h264.c.
> > > > > 
> > > > > Here are some simple first steps:
> > > > > 
> > > > > - Move a table that is only used there to svq3.c.
> > > > 
> > > > there is
> > > > static const uint8_t slice_type_map[5]= {FF_P_TYPE, FF_B_TYPE, FF_I_TYPE, FF_SP_TYPE, FF_SI_TYPE};
> > > > in h264.c
> > > 
> > > This is indeed a duplicate of golomb_to_pict_type.  So what do you want
> > > done?
> > 
> > id make the code in h264.c use the table ...
> > less code duplication ...
> 
> OK, patch attached.

ok

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081124/961910d6/attachment.pgp>



More information about the ffmpeg-devel mailing list