[PATCH] fonts: Fix typo in async ListFonts logic

Peter Hutterer peter.hutterer at who-t.net
Mon Dec 6 19:27:33 PST 2010


On Mon, Dec 06, 2010 at 06:06:57PM -0500, Adam Jackson wrote:
> This was introduced in 3ab6cd31cbdf8095b2948034fce5fb645422d8da.  Mea
> culpa.  This logic is still incorrect [1], but at least it's less
> incorrect.
> 
> [1] - https://bugzilla.redhat.com/658587
> 
> Signed-off-by: Adam Jackson <ajax at redhat.com>
> ---
>  dix/dixfonts.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/dix/dixfonts.c b/dix/dixfonts.c
> index e22eabe..cf70075 100644
> --- a/dix/dixfonts.c
> +++ b/dix/dixfonts.c
> @@ -667,7 +667,7 @@ doListFontsAndAliases(ClientPtr client, LFclosurePtr c)
>  		    ((pointer) c->client, fpe, &name, &namelen, &tmpname,
>  		     &resolvedlen, c->current.private);
>  		if (err == Suspended) {
> -		    if (ClientIsAsleep(client))
> +		    if (!ClientIsAsleep(client))
>  			ClientSleep(client,
>  				    (ClientSleepProcPtr)doListFontsAndAliases,
>  				    c);
> -- 
> 1.7.3.1

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
  Peter


More information about the xorg-devel mailing list