[FFmpeg-devel] Visibility implementation

[Diego Biurrun]

On Fri, Oct 24, 2008 at 10:30:55AM +0200, Diego Biurrun wrote:
> On Thu, Oct 23, 2008 at 09:25:35PM +0100, M?ns Rullg?rd wrote:
> > Diego Biurrun <diego at biurrun.de> writes:
> > >
> > > Updated version attached.
> > >
> > > --- libavutil/internal.h	(revision 15672)
> > > +++ libavutil/internal.h	(working copy)
> > > @@ -50,6 +50,18 @@
> > >  
> > > +#ifdef CONFIG_VISIBILITY
> > 
> > Where do you expect this to get set?  Is there any reason to disable
> > it when the compiler supports it?  I would expect to see some compiler
> > version ifdeffery or a configure check setting HAVE_ATTR_VISIBILITY or
> > similar.
> 
> ATM it would have to be set manually in config.h, i.e. configure support
> is still missing.  I'll see if I implement it in the next round.

OK, here is the next version, along with configure support.  This should
be fit for committing.

Diego
-------------- next part --------------
A non-text attachment was scrubbed...
Name: visibility.diff
Type: text/x-diff
Size: 43276 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081027/a638a74e/attachment.diff>