[PATCH libXext] Don't export xgeExtRegister
Julien Cristau
jcristau at debian.org
Sun Nov 29 15:10:37 PST 2009
On Mon, Nov 30, 2009 at 08:56:58 +1000, Peter Hutterer wrote:
> xgeExtRegister is defined here but declared in extutil.c.
>
> the gcc docs recommend adding attributes to the declaration:
> "The visibility attribute should be applied only to declarations which would
> otherwise have external linkage. [...]"
> http://gcc.gnu.org/onlinedocs/gcc/Function-Attributes.html#Function-Attributes
>
> I think it'd be better to move the attribute to extutil.c then. given that
> change,
> Acked-by: Peter Hutterer <peter.hutterer at who-t.net>
>
I've made that change and pushed. Thanks.
Cheers,
Julien
More information about the xorg-devel
mailing list