[FFmpeg-devel] Visibility implementation
Måns Rullgård
mans
Tue Oct 28 17:59:24 CET 2008
Diego Biurrun wrote:
> 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.
I don't like --enable-visiblity. Especially for something video-related
its meaning is not at all clear. Call it --enable-symbol-hiding or
similar instead.
You're still adding insane amounts of whitespace around the section
start/stop lines. Why?
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list