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

[Diego Biurrun]

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.

Diego
-------------- next part --------------
A non-text attachment was scrubbed...
Name: slice_type.diff
Type: text/x-diff
Size: 820 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081124/0f1955a7/attachment.diff>