[FFmpeg-devel] [PATCH] split mdec.c from mpeg12.c
Tue Aug 19 11:49:55 CEST 2008
On Tue, Aug 19, 2008 at 03:32:32AM +0200, Michael Niedermayer wrote:
> On Tue, Aug 19, 2008 at 03:22:28AM +0200, Diego Biurrun wrote:
> > On Tue, Aug 19, 2008 at 02:20:56AM +0200, Diego Biurrun wrote:
> > > Here is a little refactoring patch that moves mdec.c to a proper
> > > separate file that can be conditionally compiled instead of #including
> > > it from mpeg12.c.
> > >
> > > --- libavcodec/mpeg12.c (revision 14829)
> > > +++ libavcodec/mpeg12.c (working copy)
> > > @@ -152,7 +151,7 @@
> > >
> > > -static void init_vlcs(void)
> > > +void ff_init_vlcs(void)
> > Should this function be marked av_cold (in a separate patch)?
> > > @@ -620,7 +619,7 @@
> > >
> > > -static inline int decode_dc(GetBitContext *gb, int component)
> > > +inline int ff_decode_dc(GetBitContext *gb, int component)
> > I am being told that non-static inline functions can be problematic.
> > There are two possible solutions:
> > - Move the function to a header file,
> yes, but note this could end up rather messy and if it requires
> 100 things to be made global then i would prefer to keep
> #including mdec.c
See attached patch, it is rather simple. If you prefer I could split
out the rename upon commit.
> > - remove the inline keyword.
> > Aurelien did the latter when splitting off wmv2.c from msmpeg4.c.
> Was this benchmarked?
Yes, this was discussed with you back then. I ran some benchmarks,
as did Aurelien:
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 4740 bytes
Desc: not available
More information about the ffmpeg-devel