Added logging priorities and use SDLs logging facilities

By using SDL logging, we can rely on SDL to filter the log messages by
their priority before they get sent to our custom output function. For
example, we now no longer get entries for created and destroyed Window
instances by default, since these have only "debug" priority whereas the
default priority for the "app" category is "info".

We can also get log messages raised by SDL itself in our log now. Log
levels can be controlled per category through the "SDL_LOGGING"
environment variable.

Many existing log messages have been assigned a category based on their
existing prefix.

1 files changed, 9 insertions, 9 deletions

diff --git a/src/net/download.cpp b/src/net/download.cpp
index 7aab3b2f..571af7a4 100644
--- a/src/net/download.cpp
+++ b/src/net/download.cpp
@@ -108,13 +108,13 @@ bool Download::start()
 {
     assert(!mThread);   // Download already started
 
-    logger->log("Starting download: %s", mUrl.c_str());
+    Log::info("Starting download: %s", mUrl.c_str());
 
     mThread = SDL_CreateThread(downloadThread, "Download", this);
 
     if (!mThread)
     {
-        logger->log("%s", DOWNLOAD_ERROR_MESSAGE_THREAD);
+        Log::info("%s", DOWNLOAD_ERROR_MESSAGE_THREAD);
         strncpy(mError, DOWNLOAD_ERROR_MESSAGE_THREAD, CURL_ERROR_SIZE - 1);
         mState.lock()->status = DownloadStatus::Error;
         return false;
@@ -125,7 +125,7 @@ bool Download::start()
 
 void Download::cancel()
 {
-    logger->log("Canceling download: %s", mUrl.c_str());
+    Log::info("Canceling download: %s", mUrl.c_str());
     mCancel = true;
 }
 
@@ -186,7 +186,7 @@ int Download::downloadThread(void *ptr)
         if (!curl)
             break;
 
-        logger->log("Downloading: %s", d->mUrl.c_str());
+        Log::info("Downloading: %s", d->mUrl.c_str());
 
         curl_easy_setopt(curl, CURLOPT_FOLLOWLOCATION, 1);
         curl_easy_setopt(curl, CURLOPT_HTTPHEADER, d->mHeaders);
@@ -236,8 +236,8 @@ int Download::downloadThread(void *ptr)
 
         if (res != CURLE_OK)
         {
-            logger->log("curl error %d: %s host: %s",
-                        res, d->mError, d->mUrl.c_str());
+            Log::info("curl error %d: %s host: %s",
+                      res, d->mError, d->mUrl.c_str());
 
             if (file)
             {
@@ -262,9 +262,9 @@ int Download::downloadThread(void *ptr)
 
                     // Remove the corrupted file
                     ::remove(outFilename.c_str());
-                    logger->log("Checksum for file %s failed: (%lx/%lx)",
-                        d->mFileName.c_str(),
-                        adler, *d->mAdler);
+                    Log::info("Checksum for file %s failed: (%lx/%lx)",
+                              d->mFileName.c_str(),
+                              adler, *d->mAdler);
 
                     continue; // Bail out here to avoid the renaming
                 }