refactor: Make error() return type void

This is needed for the next commit to compile.

src/logging.h

@@ -264,10 +264,9 @@ static inline void LogPrintf_(const std::string& logging_function, const std::st
 #define LogPrint(category, ...)  LogDebug(category, __VA_ARGS__)
 
 template <typename... Args>
-bool error(const char* fmt, const Args&... args)
+void error(const char* fmt, const Args&... args)
 {
     LogPrintf("ERROR: %s\n", tfm::format(fmt, args...));
-    return false;
 }
 
 #endif // BITCOIN_LOGGING_H

src/node/blockstorage.cpp

@@ -1102,9 +1102,10 @@ bool BlockManager::ReadRawBlockFromDisk(std::vector<uint8_t>& block, const FlatF
         filein >> blk_start >> blk_size;
 
         if (blk_start != GetParams().MessageStart()) {
-            return error("%s: Block magic mismatch for %s: %s versus expected %s", __func__, pos.ToString(),
+            error("%s: Block magic mismatch for %s: %s versus expected %s", __func__, pos.ToString(),
                          HexStr(blk_start),
                          HexStr(GetParams().MessageStart()));
+            return false;
         }
 
         if (blk_size > MAX_SIZE) {