Jon TURNEY <[email protected]> writes:

>       os/utils.c: Fix compilation of OsBlockSIGIO when SIGIO isn't defined

This should probably look like this instead, to avoid compilers
complaining about unreachable code:

diff --git a/os/utils.c b/os/utils.c
index 3c520ad..0d77b84 100644
--- a/os/utils.c
+++ b/os/utils.c
@@ -1227,10 +1227,10 @@ OsBlockSIGIO(void)
         sigprocmask(SIG_BLOCK, &set, &PreviousSigIOMask);
         ret = sigismember(&PreviousSigIOMask, SIGIO);
         return ret;
-    } else
-        return 1;
+    }
 #endif
 #endif
+    return 1;
 }
 
 void

> Yaakov Selkowitz (2):
>       hw/xwin: use raise instead of signal

This should be fine (it's in POSIX.1-2001 according to the Linux
raise(3) man page). If we find a system this breaks on, it will be easy
to fix (testing for raise in the c library and using it only if present).

The other patches were also Reviewed-by: Keith Packard <[email protected]>

-- 
[email protected]

Attachment: pgpdn3lk2D6VO.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to