refactor: Fix CAddrMan::Check style

This change improves readability, and follows Developer Notes.

src/addrman.h

@@ -716,11 +716,10 @@ private:
     void Check()
     {
 #ifdef DEBUG_ADDRMAN
-        {
+        LOCK(cs);
-            LOCK(cs);
+        const int err = Check_();
-            int err;
+        if (err) {
-            if ((err=Check_()))
+            LogPrintf("ADDRMAN CONSISTENCY CHECK FAILED!!! err=%i\n", err);
-                LogPrintf("ADDRMAN CONSISTENCY CHECK FAILED!!! err=%i\n", err);
         }
 #endif
     }