Removed a long time warning compiling with recent GCC on Linux

diff --git a/TODO b/TODO
index 95034ea20d4a9e1fcda674f71afdacd818decdb9..4699db78205ba6e076274bad08617f88a04f2a8d 100644 (file)
--- a/TODO
+++ b/TODO
@@ -4,7 +4,7 @@ VERSION 1.1 TODO
 * Man pages for MSET MSETNX and SRANDMEMBER, Z-commands, ...
 * ZSETs missing stuff: ZINCRBY
 * Use strcoll() to compare objects in sorted sets, like it already happens for SORT.
 * Man pages for MSET MSETNX and SRANDMEMBER, Z-commands, ...
 * ZSETs missing stuff: ZINCRBY
 * Use strcoll() to compare objects in sorted sets, like it already happens for SORT.

-* EXPIRE, EXPIREAT, ZSCORE tests.
+* LPOPPUSH, EXPIRE, EXPIREAT, ZSCORE, SRANDMEMBER tests.

 * Write docs for the "STORE" operaiton of SORT, and GET "#" option.
 * Append only mode: testing and a command to rebuild the log from scratch.
 * Profiling and optimizations. For instance the commands lookup is probably starting to eat too CPU being a simple list. To implement binary search or an hash table lookup can be a win probably.
 * Write docs for the "STORE" operaiton of SORT, and GET "#" option.
 * Append only mode: testing and a command to rebuild the log from scratch.
 * Profiling and optimizations. For instance the commands lookup is probably starting to eat too CPU being a simple list. To implement binary search or an hash table lookup can be a win probably.

diff --git a/redis.c b/redis.c
index e6da5f85cd8e8e518f9d5ba4889fed696bc169a4..90063525a199ace4d74137b9007147f77051e16c 100644 (file)
--- a/redis.c
+++ b/redis.c
@@ -1893,7 +1893,9 @@ static void acceptHandler(aeEventLoop *el, int fd, void *privdata, int mask) {
         char *err = "-ERR max number of clients reached\r\n";
 
         /* That's a best effort error message, don't check write errors */
         char *err = "-ERR max number of clients reached\r\n";
 
         /* That's a best effort error message, don't check write errors */

-        (void) write(c->fd,err,strlen(err));
+        if (write(c->fd,err,strlen(err)) == -1) {
+            /* Nothing to do, Just to avoid the warning... */
+        }

         freeClient(c);
         return;
     }
         freeClient(c);
         return;
     }