Module: sems Branch: master Commit: 9701c5ed53d5b1b3d1489a60cf269b9b9a6ed071 URL: https://github.com/sems-server/sems/commit/9701c5ed53d5b1b3d1489a60cf269b9b9a6ed071
Author: Mikko Lehto <msle...@iki.fi> Committer: Mikko Lehto <msle...@iki.fi> Date: 2015-06-12T04:55:00+03:00 apps/jsonrpc: avoid compiler warning by casting to signed --- Modified: apps/jsonrpc/RpcPeer.cpp --- Diff: https://github.com/sems-server/sems/commit/9701c5ed53d5b1b3d1489a60cf269b9b9a6ed071.diff Patch: https://github.com/sems-server/sems/commit/9701c5ed53d5b1b3d1489a60cf269b9b9a6ed071.patch --- diff --git a/apps/jsonrpc/RpcPeer.cpp b/apps/jsonrpc/RpcPeer.cpp index eb5ad6d..13c003d 100644 --- a/apps/jsonrpc/RpcPeer.cpp +++ b/apps/jsonrpc/RpcPeer.cpp @@ -238,11 +238,11 @@ int JsonrpcNetstringsConnection::netstringsRead() { return REMOVE; } - if ((rcvd_size<0 && errno == EAGAIN) || - (rcvd_size<0 && errno == EWOULDBLOCK)) + if (((ssize_t)rcvd_size<0 && errno == EAGAIN) || + ((ssize_t)rcvd_size<0 && errno == EWOULDBLOCK)) return CONTINUE; // necessary? - if (rcvd_size<0) { + if ((ssize_t)rcvd_size<0) { INFO("socket error on connection [%p/%d]: %s\n", this, fd, strerror(errno)); close(); @@ -296,7 +296,7 @@ int JsonrpcNetstringsConnection::netstringsRead() { } int JsonrpcNetstringsConnection::netstringsBlockingWrite() { - if (msg_size<0) { + if ((ssize_t)msg_size<0) { close(); return REMOVE; } @@ -327,12 +327,12 @@ int JsonrpcNetstringsConnection::netstringsBlockingWrite() { 0 #endif ); - if ((written<0 && (errno==EAGAIN || errno==EWOULDBLOCK)) || + if (((ssize_t)written<0 && (errno==EAGAIN || errno==EWOULDBLOCK)) || written==0) { usleep(SEND_SLEEP); continue; } - if (written<0) { + if ((ssize_t)written<0) { if (errno == ECONNRESET) { DBG("closing connection [%p/%d] on peer hangup\n", this, fd); close(); _______________________________________________ Semsdev mailing list Semsdev@lists.iptel.org http://lists.iptel.org/mailman/listinfo/semsdev
