aboutsummaryrefslogtreecommitdiff
path: root/backends/networking/sdl_net
diff options
context:
space:
mode:
authorAlexander Tkachev2016-07-23 12:52:27 +0600
committerAlexander Tkachev2016-08-24 16:07:55 +0600
commit15c6772ff7638e104027f7b7777180e6191841fc (patch)
tree7887ce51ef6c3d6fd7ab574aa0e7da85d47067f8 /backends/networking/sdl_net
parentd57e0c89b5b20dac247cb2f43450014d84719ba6 (diff)
downloadscummvm-rg350-15c6772ff7638e104027f7b7777180e6191841fc.tar.gz
scummvm-rg350-15c6772ff7638e104027f7b7777180e6191841fc.tar.bz2
scummvm-rg350-15c6772ff7638e104027f7b7777180e6191841fc.zip
ALL: Fix debug, warning and error usage
Added prefixes, used debug(9).
Diffstat (limited to 'backends/networking/sdl_net')
-rw-r--r--backends/networking/sdl_net/client.cpp8
-rw-r--r--backends/networking/sdl_net/localwebserver.cpp20
-rw-r--r--backends/networking/sdl_net/reader.cpp10
3 files changed, 19 insertions, 19 deletions
diff --git a/backends/networking/sdl_net/client.cpp b/backends/networking/sdl_net/client.cpp
index 0026d0e346..dab38ba5c0 100644
--- a/backends/networking/sdl_net/client.cpp
+++ b/backends/networking/sdl_net/client.cpp
@@ -62,7 +62,7 @@ void Client::open(SDLNet_SocketSet set, TCPsocket socket) {
if (set) {
int numused = SDLNet_TCP_AddSocket(set, socket);
if (numused == -1) {
- error("SDLNet_AddSocket: %s\n", SDLNet_GetError());
+ error("Client: SDLNet_AddSocket: %s\n", SDLNet_GetError());
}
}
}
@@ -79,13 +79,13 @@ bool Client::readMoreIfNeeded() {
int bytes = SDLNet_TCP_Recv(_socket, _buffer, CLIENT_BUFFER_SIZE);
if (bytes <= 0) {
- warning("Client::readHeaders recv fail");
+ warning("Client::readMoreIfNeeded: recv fail");
close();
return false;
}
if (_stream->write(_buffer, bytes) != bytes) {
- warning("failed to write() into MemoryReadWriteStream");
+ warning("Client::readMoreIfNeeded: failed to write() into MemoryReadWriteStream");
close();
return false;
}
@@ -146,7 +146,7 @@ void Client::close() {
if (_socket) {
int numused = SDLNet_TCP_DelSocket(_set, _socket);
if (numused == -1)
- error("SDLNet_DelSocket: %s\n", SDLNet_GetError());
+ error("Client: SDLNet_DelSocket: %s\n", SDLNet_GetError());
}
_set = nullptr;
}
diff --git a/backends/networking/sdl_net/localwebserver.cpp b/backends/networking/sdl_net/localwebserver.cpp
index 75dbea0d56..8a40aed697 100644
--- a/backends/networking/sdl_net/localwebserver.cpp
+++ b/backends/networking/sdl_net/localwebserver.cpp
@@ -96,14 +96,14 @@ void LocalWebserver::start() {
// Create a listening TCP socket
IPaddress ip;
if (SDLNet_ResolveHost(&ip, NULL, _serverPort) == -1) {
- error("SDLNet_ResolveHost: %s\n", SDLNet_GetError());
+ error("LocalWebserver: SDLNet_ResolveHost: %s\n", SDLNet_GetError());
}
resolveAddress(&ip);
_serverSocket = SDLNet_TCP_Open(&ip);
if (!_serverSocket) {
- warning("SDLNet_TCP_Open: %s", SDLNet_GetError());
+ warning("LocalWebserver: SDLNet_TCP_Open: %s", SDLNet_GetError());
stopTimer();
g_system->displayMessageOnOSD(_("Failed to start local webserver.\nCheck whether selected port is not used by another application and try again."));
_handleMutex.unlock();
@@ -113,12 +113,12 @@ void LocalWebserver::start() {
// Create a socket set
_set = SDLNet_AllocSocketSet(MAX_CONNECTIONS + 1); //one more for our server socket
if (!_set) {
- error("SDLNet_AllocSocketSet: %s\n", SDLNet_GetError());
+ error("LocalWebserver: SDLNet_AllocSocketSet: %s\n", SDLNet_GetError());
}
int numused = SDLNet_TCP_AddSocket(_set, _serverSocket);
if (numused == -1) {
- error("SDLNet_AddSocket: %s\n", SDLNet_GetError());
+ error("LocalWebserver: SDLNet_AddSocket: %s\n", SDLNet_GetError());
}
_handleMutex.unlock();
}
@@ -183,7 +183,7 @@ void LocalWebserver::handle() {
_handleMutex.lock();
int numready = SDLNet_CheckSockets(_set, 0);
if (numready == -1) {
- error("SDLNet_CheckSockets: %s\n", SDLNet_GetError());
+ error("LocalWebserver: SDLNet_CheckSockets: %s\n", SDLNet_GetError());
} else if (numready) {
acceptClient();
}
@@ -268,11 +268,11 @@ void LocalWebserver::resolveAddress(void *ipAddress) {
// default way (might work everywhere, surely works on Windows)
const char *name = SDLNet_ResolveIP(ip);
if (name == NULL) {
- warning("SDLNet_ResolveHost: %s\n", SDLNet_GetError());
+ warning("LocalWebserver: SDLNet_ResolveHost: %s\n", SDLNet_GetError());
} else {
IPaddress localIp;
if (SDLNet_ResolveHost(&localIp, name, _serverPort) == -1) {
- warning("SDLNet_ResolveHost: %s\n", SDLNet_GetError());
+ warning("LocalWebserver: SDLNet_ResolveHost: %s\n", SDLNet_GetError());
} else {
_address = Common::String::format(
"http://%u.%u.%u.%u:%u/",
@@ -284,7 +284,7 @@ void LocalWebserver::resolveAddress(void *ipAddress) {
// check that our trick worked
if (_address.contains("/127.0.0.1:") || _address.contains("localhost") || _address.contains("/0.0.0.0:"))
- warning("Failed to resolve IP with the default way");
+ warning("LocalWebserver: Failed to resolve IP with the default way");
else
return;
@@ -307,7 +307,7 @@ void LocalWebserver::resolveAddress(void *ipAddress) {
tmpAddrPtr = &((struct sockaddr_in *)i->ifa_addr)->sin_addr;
char addressBuffer[INET_ADDRSTRLEN];
inet_ntop(AF_INET, tmpAddrPtr, addressBuffer, INET_ADDRSTRLEN);
- debug("%s IP Address %s", i->ifa_name, addressBuffer);
+ debug(9, "%s IP Address %s", i->ifa_name, addressBuffer);
addr = addressBuffer;
}
@@ -317,7 +317,7 @@ void LocalWebserver::resolveAddress(void *ipAddress) {
tmpAddrPtr = &((struct sockaddr_in6 *)i->ifa_addr)->sin6_addr;
char addressBuffer[INET6_ADDRSTRLEN];
inet_ntop(AF_INET6, tmpAddrPtr, addressBuffer, INET6_ADDRSTRLEN);
- debug("%s IP Address %s", i->ifa_name, addressBuffer);
+ debug(9, "%s IP Address %s", i->ifa_name, addressBuffer);
addr = addressBuffer;
}
*/
diff --git a/backends/networking/sdl_net/reader.cpp b/backends/networking/sdl_net/reader.cpp
index 828f583003..0e4cc9a106 100644
--- a/backends/networking/sdl_net/reader.cpp
+++ b/backends/networking/sdl_net/reader.cpp
@@ -138,7 +138,7 @@ void readFromThatUntilLineEnd(const char *cstr, Common::String needle, Common::S
void Reader::handleFirstHeaders(Common::String headers) {
if (!_boundary.empty()) {
- warning("handleFirstHeaders() called when first headers were already handled");
+ warning("Reader: handleFirstHeaders() called when first headers were already handled");
return;
}
@@ -252,7 +252,7 @@ void Reader::parseQueryParameters() {
} else {
if (_query[i] == '&') {
if (_queryParameters.contains(key))
- warning("Query parameter \"%s\" is already set!", key.c_str());
+ warning("Reader: query parameter \"%s\" is already set!", key.c_str());
else
_queryParameters[key] = LocalWebserver::urlDecode(value);
readingKey = true;
@@ -265,7 +265,7 @@ void Reader::parseQueryParameters() {
if (!key.empty()) {
if (_queryParameters.contains(key))
- warning("Query parameter \"%s\" is already set!", key.c_str());
+ warning("Reader: query parameter \"%s\" is already set!", key.c_str());
else
_queryParameters[key] = LocalWebserver::urlDecode(value);
}
@@ -410,9 +410,9 @@ bool Reader::readBlockContent(Common::WriteStream *stream) {
if (bts == "--")
_allContentRead = true;
else if (bts != "\r\n")
- warning("strange bytes: \"%s\"", bts.c_str());
+ warning("Reader: strange bytes: \"%s\"", bts.c_str());
} else {
- warning("strange ending");
+ warning("Reader: strange ending");
_allContentRead = true;
}