[net] remove unused return type bool from CNode::Ban()

jonasschnelli · jonasschnelli · commit e8b93473f12e · 2015-06-17T21:40:55.000+02:00
diff --git a/src/net.cpp b/src/net.cpp
@@ -458,16 +458,14 @@ bool CNode::IsBanned(CNetAddr ip)
     return fResult;
 }
 
-bool CNode::Ban(const CNetAddr &addr, int64_t bantimeoffset) {
+void CNode::Ban(const CNetAddr &addr, int64_t bantimeoffset) {
     int64_t banTime = GetTime()+GetArg("-bantime", 60*60*24);  // Default 24-hour ban
     if (bantimeoffset > 0)
         banTime = GetTime()+bantimeoffset;
 
     LOCK(cs_setBanned);
     if (setBanned[addr] < banTime)
         setBanned[addr] = banTime;
-
-    return true;
 }
 
 bool CNode::Unban(const CNetAddr &addr) {
diff --git a/src/net.h b/src/net.h
@@ -606,7 +606,7 @@ class CNode
     // new code.
     static void ClearBanned(); // needed for unit testing
     static bool IsBanned(CNetAddr ip);
-    static bool Ban(const CNetAddr &ip, int64_t bantimeoffset = 0);
+    static void Ban(const CNetAddr &ip, int64_t bantimeoffset = 0);
     static bool Unban(const CNetAddr &ip);
     static void GetBanned(std::map<CNetAddr, int64_t> &banmap);
 
