[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