-Wlogical-op warning on DeliverEmulatedMotionEvent

Alan Coopersmith alan.coopersmith at oracle.com
Sat Oct 26 23:23:05 CEST 2013


gcc -Wlogical-op on Xorg git master says:

exevents.c: In function 'DeliverEmulatedMotionEvent':
exevents.c:1480:13: warning: logical 'or' of collectively exhaustive tests is 
always true [-Wlogical-op]

The relevant snippet of exevents.c:

    1479         if (ti->listeners[0].type != LISTENER_POINTER_REGULAR ||
    1480             ti->listeners[0].type != LISTENER_POINTER_GRAB)
    1481             return;

The warning seems to be correct, as they are two different values in an enum,
all values are not equal to at least one of the two, so the return is always
followed and the following 20 lines in that block become dead code.

I'd guess the || should be &&, but figured you'd be in a better position to
evaluate & test that.

-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list