Change the scraper logger to a singleton pattern

a la Bitcoin's newer approach.

Author: jamescowens

src/scraper/scraper.cpp

@@ -332,6 +332,14 @@ class logger
     }
 };
 
+logger& LogInstance()
+{
+    // This is similar to Bitcoin's newer approach.
+    static logger* scraperlogger{new logger()};
+    return *scraperlogger;
+}
+
+
 boost::gregorian::date logger::PrevArchiveCheckDate = boost::posix_time::from_time_t(GetAdjustedTime()).date();
 CCriticalSection logger::cs_log;
 
@@ -361,11 +369,13 @@ void _log(logattribute eType, const std::string& sCall, const std::string& sMess
     }
 
     sOut = tfm::format("%s [%s] <%s> : %s", DateTimeStrFormat("%x %H:%M:%S", GetAdjustedTime()), sType, sCall, sMessage);
-    logger log;
+
+    //logger log;
+    logger& log = LogInstance();
 
     log.output(sOut);
 
-    log.closelogfile();
+    //log.closelogfile();
 
     // Send to UI for log window.
     uiInterface.NotifyScraperEvent(scrapereventtypes::Log, CT_NEW, sOut);
@@ -681,6 +691,9 @@ void ScraperApplyAppCacheEntries()
 // It can also be called in "single shot" mode.
 void Scraper(bool bSingleShot)
 {
+    // Initialize log singleton.
+    LogInstance();
+
     // Initialize these while still single-threaded. They cannot be initialized during declaration because GetDataDir()
     // gives the wrong value that early. If they are already initialized then leave them alone (because this function
     // can be called in singleshot mode.
@@ -776,14 +789,14 @@ void Scraper(bool bSingleShot)
                 }
 
                 // Need the log archive check here, because we don't run housekeeping in this while loop.
-                logger log;
+                logger& log = LogInstance();
 
                 fs::path plogfile_out;
 
                 if (log.archive(false, plogfile_out))
                     _log(logattribute::INFO, "Scraper", "Archived scraper.log to " + plogfile_out.filename().string());
 
-                log.closelogfile();
+                //log.closelogfile();
 
                 sbage = SuperblockAge();
                 _log(logattribute::INFO, "Scraper", "Superblock not needed. age=" + std::to_string(sbage));
@@ -1040,14 +1053,14 @@ bool ScraperHousekeeping()
                  + ", Popularity: " + std::to_string(network_hash.second));
     }
 
-    logger log;
+    logger& log = LogInstance();
 
     fs::path plogfile_out;
 
     if (log.archive(false, plogfile_out))
         _log(logattribute::INFO, "ScraperHousekeeping", "Archived scraper.log to " + plogfile_out.filename().string());
 
-    log.closelogfile();
+    //log.closelogfile();
 
     return true;
 }
@@ -4245,12 +4258,12 @@ UniValue archivescraperlog(const UniValue& params, bool fHelp)
                 "archivescraperlog takes no arguments and results in immediate archiving of the scraper log\n"
                 );
 
-    logger log;
+    logger& log = LogInstance();
 
     fs::path pfile_out;
     bool ret = log.archive(true, pfile_out);
 
-    log.closelogfile();
+    //log.closelogfile();
 
     return UniValue(ret);
 }