[PATCH libXext] Don't export xgeExtRegister

Peter Hutterer peter.hutterer at who-t.net
Sun Nov 29 14:56:58 PST 2009


On Sat, Nov 28, 2009 at 03:38:33PM +0100, Julien Cristau wrote:
> Its only caller is XextAddDisplay()
> ---
>  src/Xge.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/Xge.c b/src/Xge.c
> index 7a583e5..2ea5d27 100644
> --- a/src/Xge.c
> +++ b/src/Xge.c
> @@ -294,7 +294,7 @@ _xgeEventToWire(Display* dpy, XEvent* re, xEvent* event)
>  /*
>   * Extensions need to register callbacks for their events.
>   */
> -Bool
> +_X_HIDDEN Bool
>  xgeExtRegister(Display* dpy, int offset, XExtensionHooks* callbacks)
>  {
>      XGEExtNode* newExt;
> -- 
> 1.6.5

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>

Cheers,
  Peter


More information about the xorg-devel mailing list