[PATCH:libICE 6/6] Remove ancient workaround for System V/386 Release 4.2 compiler bug

Matthieu Herrb matthieu.herrb at laas.fr
Thu Nov 10 01:37:25 PST 2011


On Wed, Nov 09, 2011 at 10:19:55PM -0800, Alan Coopersmith wrote:
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>

Reviewed-by: Matthieu Herrb <matthieu.herrb at laas.fr>
> ---
>  src/process.c |   12 +-----------
>  1 files changed, 1 insertions(+), 11 deletions(-)
> 
> diff --git a/src/process.c b/src/process.c
> index b566010..1e53ef9 100644
> --- a/src/process.c
> +++ b/src/process.c
> @@ -2279,17 +2279,7 @@ ProcessProtocolReply (
>  	    IcePoAuthProc authProc = myProtocol->auth_procs[(int)
>  		(iceConn->protosetup_to_you->my_auth_index)];
>  
> -#ifdef SVR4
> -
> -/*
> - * authProc is never NULL, but the cc compiler on UNIX System V/386
> - * Release 4.2 Version 1 screws up an optimization.  Unless there is
> - * some sort of reference to authProc before the function call, the
> - * function call will seg fault.
> - */
> -	    if (authProc)
> -#endif
> -		(*authProc) (iceConn,
> +	    (*authProc) (iceConn,
>  		&iceConn->protosetup_to_you->my_auth_state,
>  		True /* clean up */, False /* swap */,
>  	        0, NULL, NULL, NULL, NULL);
> -- 
> 1.7.3.2
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel

-- 
Matthieu Herrb


More information about the xorg-devel mailing list