[PATCH] Don't call pixman_disable_out_of_bounds_workaround() anymore.

Jeremy Huddleston jeremyhu at apple.com
Tue May 3 10:08:41 PDT 2011


Seems simple enough.

Is pixman_disable_out_of_bounds_workaround now just a no-op, or was the symbol removed from pixman?  Hopefully the former, but I wanted to make sure...

Reviewed-by: Jeremy Huddleston <jeremyhu at apple.com>

On May 3, 2011, at 3:36 AM, Søren Sandmann wrote:

> From: Søren Sandmann Pedersen <ssp at redhat.com>
> 
> Pixman used to have a workaround for a bug in old X servers, and this
> function was used to disable that workaround in servers known to be
> fixed.
> 
> In current versions of pixman, including the version the X server
> depends on, the workaround doesn't exist anymore, so there is no point
> disabling it.
> 
> Signed-off-by: Søren Sandmann <ssp at redhat.com>
> ---
> dix/main.c |    2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
> 
> diff --git a/dix/main.c b/dix/main.c
> index 31e2d48..702176d 100644
> --- a/dix/main.c
> +++ b/dix/main.c
> @@ -139,8 +139,6 @@ int main(int argc, char *argv[], char *envp[])
> 
>     InitRegions();
> 
> -    pixman_disable_out_of_bounds_workaround();
> -
>     CheckUserParameters(argc, argv, envp);
> 
>     CheckUserAuthorization();
> -- 
> 1.7.4
> 
> _______________________________________________
> 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



More information about the xorg-devel mailing list