[PATCH] Xext: un-export all XGE functions.

2008-12-08 Thread Peter Hutterer
These functions are only to be used by X server extensions, so let's not expose them to the world. Signed-off-by: Peter Hutterer [EMAIL PROTECTED] --- Xext/geext.h | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/Xext/geext.h b/Xext/geext.h index

Re: [PATCH] Xext: un-export all XGE functions.

2008-12-08 Thread Paulo César Pereira de Andrade
These functions are only to be used by X server extensions, so let's not expose them to the world. This will not work as expected if the X Server is compiled with hidden symbols. For the moment at least, it must be specified as both extern _X_EXPORT (what may be counter intuitive I agree).

Re: [PATCH] Xext: un-export all XGE functions.

2008-12-08 Thread Peter Hutterer
On Mon, Dec 08, 2008 at 09:20:57AM -0800, Alan Coopersmith wrote: Peter Hutterer wrote: These functions are only to be used by X server extensions, so let's not expose them to the world. What about extensions in loadable modules? good point. tbh the main reason for this patch is that I don't

Re: [PATCH] Xext: un-export all XGE functions.

2008-12-08 Thread Peter Hutterer
On Mon, Dec 08, 2008 at 07:09:13PM -0200, Paulo César Pereira de Andrade wrote: These functions are only to be used by X server extensions, so let's not expose them to the world. This will not work as expected if the X Server is compiled with hidden symbols. For the moment at least,

Re: [PATCH] Xext: un-export all XGE functions.

2008-12-08 Thread Paulo César Pereira de Andrade
Peter Hutterer wrote: On Mon, Dec 08, 2008 at 07:09:13PM -0200, Paulo César Pereira de Andrade wrote: These functions are only to be used by X server extensions, so let's not expose them to the world. This will not work as expected if the X Server is compiled with hidden symbols.

Re: [PATCH] Xext: un-export all XGE functions.

2008-12-08 Thread Daniel Stone
Hi, On Mon, Dec 08, 2008 at 10:44:52PM -0200, Paulo César Pereira de Andrade wrote: I will try to do as small as possible reorganization on the sdk headers, and add private and not installed headers. Please run these past the list first. Cheers, Daniel signature.asc Description: Digital