[PATCH 1/3] OtherClientGone: Remove unreachable return statement

Alan Coopersmith alan.coopersmith at oracle.com
Wed Jun 13 22:06:04 PDT 2012


Now that FatalError is marked as _X_NORETURN, the compilers know we
can't get here, and the return statement added to make them happy in
the past now makes them unhappy.

Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
---
 dix/events.c |    1 -
 1 file changed, 1 deletion(-)

diff --git a/dix/events.c b/dix/events.c
index 83ae5c9..069888b 100644
--- a/dix/events.c
+++ b/dix/events.c
@@ -4313,7 +4313,6 @@ OtherClientGone(pointer value, XID id)
         prev = other;
     }
     FatalError("client not on event list");
-     /*NOTREACHED*/ return -1;  /* make compiler happy */
 }
 
 int
-- 
1.7.9.2



More information about the xorg-devel mailing list