[Ffmpeg-devel] [PATCH] last move of functions in vorbis.c to vorbis.h

[Aurelien Jacobs]

On Sun, 1 Oct 2006 14:00:12 +0200
Oded Shimon <ods15 at ods15.dyndns.org> wrote:

> On Sun, Oct 01, 2006 at 01:54:08PM +0200, Michael Niedermayer wrote:
> > Hi
> > 
> > On Sun, Oct 01, 2006 at 01:39:50PM +0200, Oded Shimon wrote:
> > > last patch necessary for all shared functions (that I noticed).
> > > One unfortunate thing is I had to remove some debugging printfs from the 
> > > floor function.
> > > 
> > > render_line() is called quite a bit and during decode, so I made it static 
> > > inline in vorbis.h, gcc refuses to inline it though.
> > > 
> > > Any objections to commit?
> > 
> > looks ok
> 
> Committed with one change, without the inline keyword to avoid annoying 
> warnings... they were given for vorbis_enc.c as well.

Hum... without the inline keyword it shouldn't go in the .h file IMO.
It should be shared at the object level...

Aurel