[PATCH] rootless: Remove ROOTLESS_WORKAROUND

Jon TURNEY jon.turney at dronecode.org.uk
Thu Aug 26 05:53:30 PDT 2010


On 16/08/2010 04:11, Jeremy Huddleston wrote:
>
> This was already removed for XWin (20701522be803fe47e921fcf059dadf64c7f287d)
> with no reported side effects.  XQuartz seems to be behaving ok without it as
> well.  While this possibly brings back bug #1168, we don't have any reproduction
> steps for that issue, and if it crops up again, we should fix it a real way
> rather than this hokey workaround which doesn't even work for COMPOSITE.
>
> Signed-off-by: Jeremy Huddleston<jeremyhu at apple.com>

Reviewed-by: Jon TURNEY <jon.turney at dronecode.org.uk>

You might wish to note in the commit message that this is effectively a revert 
of commits b2135e58 and d7fef522

> ---
>   configure.ac          |    2 +-
>   miext/damage/damage.c |    6 +-----
>   2 files changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 1f4eb34..7db4437 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -1926,7 +1926,7 @@ if test "x$XQUARTZ" = xyes; then
>
>   	AC_CHECK_LIB([Xplugin],[xp_init],[:])
>
> -	CFLAGS="${CFLAGS} -DROOTLESS_WORKAROUND -DROOTLESS_SAFEALPHA -DNO_ALLOCA"
> +	CFLAGS="${CFLAGS} -DROOTLESS_SAFEALPHA -DNO_ALLOCA"
>
>   	PKG_CHECK_MODULES(XPBPROXY, $APPLEWMPROTO $LIBAPPLEWM xfixes x11)
>
> diff --git a/miext/damage/damage.c b/miext/damage/damage.c
> index 1cf0513..e0e96f2 100644
> --- a/miext/damage/damage.c
> +++ b/miext/damage/damage.c
> @@ -89,11 +89,7 @@ getDrawableDamageRef (DrawablePtr pDrawable)
>   	ScreenPtr   pScreen = pDrawable->pScreen;
>
>   	pPixmap = 0;
> -	if (pScreen->GetWindowPixmap
> -#ifdef ROOTLESS_WORKAROUND
> -	&&  ((WindowPtr)pDrawable)->viewable
> -#endif
> -	    )
> +	if (pScreen->GetWindowPixmap)
>   	    pPixmap = (*pScreen->GetWindowPixmap) ((WindowPtr)pDrawable);
>
>   	if (!pPixmap)



More information about the xorg-devel mailing list