[compiz] Bug in otherScreenGrabExist?

Mike Dransfield mike at blueroot.co.uk
Thu May 3 16:49:31 PDT 2007


I am fairly sure that this is just a bug in the codes logic, but
I thought I should check first

I noticed that the function was always returning true if only
one grab is checked for (maybe other cases too)

If nobody shouts I shall commit it because the behaviour in
my plugin is now as expected.


-------------- next part --------------
diff --git a/src/screen.c b/src/screen.c
index 7dcc452..959b72f 100644
--- a/src/screen.c
+++ b/src/screen.c
@@ -2427,7 +2427,7 @@ otherScreenGrabExist (CompScreen *s, ...)
 
 	    va_end (ap);
 
-	    if (!name)
+	    if (name)
 		return TRUE;
 	}
     }


More information about the compiz mailing list