Fix several LogPrint related variadic warnings from linter

gridcoin-community · Mar 27, 2022 · 2e77aef · 2e77aef
1 parent 0863ea3
commit 2e77aef
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 13 deletions.
diff --git a/src/gridcoin/scraper/scraper_net.cpp b/src/gridcoin/scraper/scraper_net.cpp
@@ -563,8 +563,8 @@ EXCLUSIVE_LOCKS_REQUIRED(CScraperManifest::cs_mapManifest)
                                                                                                iter->second)));
             if (!iter2.second)
             {
-                LogPrint("WARN: %s: Manifest insertion attempt into pending deleted map failed because an entry with the same "
-                         "hash = %s, already exists. This should not happen.", __func__, nHash.GetHex());
+                LogPrintf("WARN: %s: Manifest insertion attempt into pending deleted map failed because an entry with the same "
+                          "hash = %s, already exists. This should not happen.", __func__, nHash.GetHex());
             }
             else
             {
@@ -605,8 +605,8 @@ CScraperManifest::DeleteManifest(std::map<uint256, std::shared_ptr<CScraperManif
                                                                                            iter->second)));
         if (!iter2.second)
         {
-            LogPrint("WARN: %s: Manifest insertion attempt into pending deleted map failed because an entry with the same "
-                     "hash = %s, already exists. This should not happen.", __func__, iter->first.GetHex());
+            LogPrintf("WARN: %s: Manifest insertion attempt into pending deleted map failed because an entry with the same "
+                      "hash = %s, already exists. This should not happen.", __func__, iter->first.GetHex());
         }
         else
         {

diff --git a/src/main.cpp b/src/main.cpp
@@ -3263,21 +3263,26 @@ void PrintBlockTree() EXCLUSIVE_LOCKS_REQUIRED(cs_main)
         // print split or gap
         if (nCol > nPrevCol)
         {
-            for (int i = 0; i < nCol-1; i++)
+            for (int i = 0; i < nCol-1; i++) {
                 LogPrintf("| ");
+            }
+
             LogPrintf("|\\");
         }
         else if (nCol < nPrevCol)
         {
-            for (int i = 0; i < nCol; i++)
+            for (int i = 0; i < nCol; i++) {
                 LogPrintf("| ");
+            }
+
             LogPrintf("|");
        }
         nPrevCol = nCol;
 
         // print columns
-        for (int i = 0; i < nCol; i++)
+        for (int i = 0; i < nCol; i++) {
             LogPrintf("| ");
+        }
 
         // print item
         CBlock block;

diff --git a/src/miner.cpp b/src/miner.cpp
@@ -934,7 +934,8 @@ void SplitCoinStakeOutput(CBlock &blocknew, int64_t &nReward, bool &fEnableStake
 
             if (dSumAllocation + iterSideStake->second > 1.0)
             {
-                LogPrintf("WARN: SplitCoinStakeOutput: allocation percentage over 100\%, ending sidestake allocations.");
+                LogPrintf("WARN: SplitCoinStakeOutput: allocation percentage over 100 percent, "
+                          "ending sidestake allocations.");
                 break;
             }
 
@@ -1327,7 +1328,7 @@ SideStakeAlloc GetSideStakingStatusAndAlloc()
         dSumAllocation += dAllocation;
         if (dSumAllocation > 1.0)
         {
-            LogPrintf("WARN: %s: allocation percentage over 100\%, ending sidestake allocations.", __func__);
+            LogPrintf("WARN: %s: allocation percentage over 100 percent, ending sidestake allocations.", __func__);
             break;
         }
 

diff --git a/src/util/time.cpp b/src/util/time.cpp
@@ -261,7 +261,7 @@ int64_t MilliTimer::GetStartTime(const std::string& label)
         internal_timer = timer_map.at(label);
     }
     catch (std::out_of_range&) {
-        LogPrintf("WARNING: %s: Timer with specified label does not exist. Returning zero start time.");
+        LogPrintf("WARNING: %s: Timer with specified label does not exist. Returning zero start time.", __func__);
     }
 
     return internal_timer.start_time;
@@ -291,7 +291,7 @@ const MilliTimer::timer MilliTimer::GetTimes(const std::string& log_string, cons
     }
     catch (std::out_of_range&)
     {
-        LogPrintf("WARNING: %s: Timer with specified label does not exist. Returning zeroed timer.");
+        LogPrintf("WARNING: %s: Timer with specified label does not exist. Returning zeroed timer.", __func__);
         timer = {};
         return timer;
     }
@@ -300,8 +300,8 @@ const MilliTimer::timer MilliTimer::GetTimes(const std::string& log_string, cons
     // minimize lock time.
     if (internal_timer.log)
     {
-        LogPrintf("timer %s: %s: elapsed time: %" PRId64 " ms, time since last check: %" PRId64 " ms.",
-                  label, log_string, timer.elapsed_time, timer.time_since_last_check);
+        LogPrintf("INFO: %s: timer %s: %s: elapsed time: %" PRId64 " ms, time since last check: %" PRId64 " ms.",
+                  __func__, label, log_string, timer.elapsed_time, timer.time_since_last_check);
     }
 
     return timer;