<div dir="ltr">Reviewed-by: Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net">jstpierre@mecheye.net</a>></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 7, 2014 at 7:02 PM, Alan Coopersmith <span dir="ltr"><<a href="mailto:alan.coopersmith@oracle.com" target="_blank">alan.coopersmith@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Solaris <sys/errno.h> has:<br>
 #define EWOULDBLOCK       EAGAIN<br>
so checking (errno == EAGAIN || errno == EWOULDBLOCK) is overkill.<br>
<br>
This leads cppcheck 1.62 to complain:<br>
[FSlibInt.c:153] -> [FSlibInt.c:153]: (style) Same expression on both sides of '||'.<br>
[FSlibInt.c:301] -> [FSlibInt.c:301]: (style) Same expression on both sides of '||'.<br>
[FSlibInt.c:379] -> [FSlibInt.c:379]: (style) Same expression on both sides of '||'.<br>
[FSlibInt.c:472] -> [FSlibInt.c:472]: (style) Same expression on both sides of '||'.<br>
<br>
This quiets it, and reduces the number of calls Solaris Studio cc<br>
generates to the __errno() function to get the thread-specific errno value.<br>
<br>
Signed-off-by: Alan Coopersmith <<a href="mailto:alan.coopersmith@oracle.com">alan.coopersmith@oracle.com</a>><br>
---<br>
 src/FSlibInt.c |    5 ++++-<br>
 1 file changed, 4 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/FSlibInt.c b/src/FSlibInt.c<br>
index 0fabc96..cb53e44 100644<br>
--- a/src/FSlibInt.c<br>
+++ b/src/FSlibInt.c<br>
@@ -66,11 +66,14 @@ static const char * _SysErrorMsg ( int n );<br>
<br>
 /* check for both EAGAIN and EWOULDBLOCK, because some supposedly POSIX<br>
  * systems are broken and return EWOULDBLOCK when they should return EAGAIN<br>
+ *<br>
+ * Solaris defines EWOULDBLOCK to be EAGAIN, so don't need to check twice<br>
+ * for it.<br>
  */<br>
 #ifdef WIN32<br>
 #define ETEST() (WSAGetLastError() == WSAEWOULDBLOCK)<br>
 #else<br>
-#if defined(EAGAIN) && defined(EWOULDBLOCK)<br>
+#if defined(EAGAIN) && defined(EWOULDBLOCK) && (EAGAIN != EWOULDBLOCK)<br>
 #define ETEST() (errno == EAGAIN || errno == EWOULDBLOCK)<br>
 #else<br>
 #ifdef EAGAIN<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.9.2<br>
<br>
_______________________________________________<br>
<a href="mailto:xorg-devel@lists.x.org">xorg-devel@lists.x.org</a>: X.Org development<br>
Archives: <a href="http://lists.x.org/archives/xorg-devel" target="_blank">http://lists.x.org/archives/xorg-devel</a><br>
Info: <a href="http://lists.x.org/mailman/listinfo/xorg-devel" target="_blank">http://lists.x.org/mailman/listinfo/xorg-devel</a><br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>  Jasper<br>
</div>