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

[Diego Biurrun]

On Mon, Nov 24, 2008 at 04:29:30PM +0100, Michael Niedermayer wrote:
> 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

Applied.

Diego