From f32c8b4f2185833242d231681d5b4e41f00da1f3 Mon Sep 17 00:00:00 2001 From: DarthArgus Date: Sun, 31 Jan 2016 12:51:49 -0600 Subject: [PATCH] theoretically this will fix our unsigned comparison warnings --- .../CSAssist/utils/TcpLibrary/TcpConnection.cpp | 8 ++++---- .../soePlatform/CSAssist/utils/TcpLibrary/TcpManager.cpp | 6 +++--- external/ours/library/archive/src/shared/ByteStream.cpp | 2 +- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpConnection.cpp b/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpConnection.cpp index 0b3366a8..97ca7fd1 100755 --- a/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpConnection.cpp +++ b/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpConnection.cpp @@ -58,7 +58,7 @@ TcpConnection::TcpConnection(TcpManager *tcpManager, TcpBlockAllocator *sendAllo int err = connect(m_socket, (sockaddr *)&m_addr, sizeof(m_addr)); - if(err == SOCKET_ERROR) + if((unsigned) err == SOCKET_ERROR) { #ifdef WIN32 int sockerr = WSAGetLastError(); @@ -132,7 +132,7 @@ TcpConnection::TcpConnection(TcpManager *tcpManager, TcpBlockAllocator *sendAllo void TcpConnection::setOptions() { - if (m_socket != INVALID_SOCKET) + if ((unsigned) m_socket != INVALID_SOCKET) { #if defined(WIN32) unsigned long isNonBlocking = 1; @@ -249,7 +249,7 @@ int TcpConnection::finishConnect() #else // not WIN32 int err = connect(m_socket, (sockaddr *)&m_addr, sizeof(m_addr)); - if(err == SOCKET_ERROR) + if((unsigned) err == SOCKET_ERROR) { if (errno != EINPROGRESS && errno != EALREADY) { @@ -413,7 +413,7 @@ void TcpConnection::Disconnect(bool notifyApplication) } - if(m_socket != INVALID_SOCKET) + if((unsigned) m_socket != INVALID_SOCKET) { #if defined(WIN32) closesocket(m_socket); diff --git a/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpManager.cpp b/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpManager.cpp index fa4d61a4..29d51de0 100755 --- a/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpManager.cpp +++ b/external/3rd/library/soePlatform/CSAssist/utils/TcpLibrary/TcpManager.cpp @@ -101,7 +101,7 @@ bool TcpManager::BindAsServer() m_socket = socket(AF_INET, SOCK_STREAM, 0); - if (m_socket != INVALID_SOCKET) + if ((unsigned) m_socket != INVALID_SOCKET) { #if defined(WIN32) FD_SET(m_socket, &m_permfds);//the socket this server is listening on @@ -197,7 +197,7 @@ TcpConnection *TcpManager::acceptClient() SOCKET sock = ::accept(m_socket, (sockaddr *) &addr, (socklen_t *) &addrLength); - if (sock != INVALID_SOCKET) + if ((unsigned) sock != INVALID_SOCKET) { newConn = new TcpConnection(this, &m_allocator, m_params, sock, IPAddress(addr.sin_addr.s_addr), ntohs(addr.sin_port)); addNewConnection(newConn); @@ -445,7 +445,7 @@ bool TcpManager::GiveTime(unsigned maxTimeAcceptingConnections,unsigned maxSendT int cnt = poll(pollfds, numfds, 1); - if(cnt == SOCKET_ERROR) + if((unsigned) cnt == SOCKET_ERROR) { //poll not working? //TODO: need to notify client somehow, don't think we can assume a fatal error here diff --git a/external/ours/library/archive/src/shared/ByteStream.cpp b/external/ours/library/archive/src/shared/ByteStream.cpp index da544b94..2e19cb81 100755 --- a/external/ours/library/archive/src/shared/ByteStream.cpp +++ b/external/ours/library/archive/src/shared/ByteStream.cpp @@ -345,7 +345,7 @@ ByteStream::Data *ByteStream::Data::getNewData() void ByteStream::Data::releaseOldData(ByteStream::Data *oldData) { - assert(reinterpret_cast(oldData) != 0xefefefefu); + assert((unsigned) reinterpret_cast(oldData) != 0xefefefefu); if (oldData->size > 4096) delete oldData;