[PATCH] Replace 'pointer' with explicit 'void *'

Alan Coopersmith alan.coopersmith at oracle.com
Mon Jan 13 16:34:41 PST 2014


On 01/13/14 04:19 PM, Keith Packard wrote:
> To get rid of the 'pointer' typedef, stop using it locally. That way,
> when _XTYPEDEF_POINTER is defined before Xdefs.h is included, it won't
> get defined and Xdefs.h will still compile.
>
> Signed-off-by: Keith Packard <keithp at keithp.com>
> ---
>   Xdefs.h | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/Xdefs.h b/Xdefs.h
> index 46ffdad..e25a208 100644
> --- a/Xdefs.h
> +++ b/Xdefs.h
> @@ -101,8 +101,8 @@ typedef FSID AccContext;
>   typedef struct timeval **OSTimePtr;
>
>
> -typedef void (* BlockHandlerProcPtr)(pointer /* blockData */,
> +typedef void (* BlockHandlerProcPtr)(void * /* blockData */,
>   				     OSTimePtr /* pTimeout */,
> -				     pointer /* pReadmask */);
> +				     void * /* pReadmask */);

Does this need to be #ifdef'ed to avoid breaking people building
1.15.x & earlier releases?

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list