[FFmpeg-devel] [PATCH] Remove swscale_internal.h:fmt_depth()

Michael Niedermayer michaelni
Sun Jan 17 22:04:47 CET 2010


On Sun, Jan 17, 2010 at 06:46:40PM +0100, Stefano Sabatini wrote:
> On date Saturday 2010-01-16 17:08:48 -0200, Ramiro Polla encoded:
> > Hi,
> > 
> > On Sat, Jan 16, 2010 at 4:59 PM, Stefano Sabatini
> > <stefano.sabatini-lala at poste.it> wrote:
> > > Hi, I'm aware this patch introduces a slow-down, an idea would be to
> > > initialize a ff_bits_per_pixel array during the init phase, and then
> > > use a function of the kind:
> > >
> > > static inline int fmt_depth(int fmt)
> > > {
> > > ? ?return ff_bits_per_pixel[fmt];
> > > }
> > >
> > > Would be that acceptable?
> > > In this case can you suggest where to initialize stuff?
> > 
> > I think all code that uses fmt_depth currently should eventually be
> > moved to some init code that's only run once, and so a small slow-down
> > wouldn't be a problem.
> 
> Check the attached: smaller, more extensible, faster, the price is a
> little more bloat in the context.
> 

> Regression test passed.

if(regression == swscale_example) patch ok
else not ok

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

I have often repented speaking, but never of holding my tongue.
-- Xenocrates
-------------- 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/20100117/334495fb/attachment.pgp>



More information about the ffmpeg-devel mailing list