Clean up

1 files changed, 113 insertions, 52 deletions

diff --git a/src/RpcMethodFactory.cc b/src/RpcMethodFactory.cc
index 7cc919de..1e505cf5 100644
--- a/src/RpcMethodFactory.cc
+++ b/src/RpcMethodFactory.cc
@@ -50,109 +50,170 @@ std::unique_ptr<RpcMethod> noSuchRpcMethod;
 } // namespace
 
 namespace {
-std::unique_ptr<RpcMethod>
-createMethod(const std::string& methodName)
+std::unique_ptr<RpcMethod> createMethod(const std::string& methodName)
 {
-  if(methodName == AddUriRpcMethod::getMethodName()) {
+  if (methodName == AddUriRpcMethod::getMethodName()) {
     return make_unique<AddUriRpcMethod>();
+  }
+
 #ifdef ENABLE_BITTORRENT
-  } else if(methodName == AddTorrentRpcMethod::getMethodName()) {
+  if (methodName == AddTorrentRpcMethod::getMethodName()) {
     return make_unique<AddTorrentRpcMethod>();
+  }
+
+  if (methodName == GetPeersRpcMethod::getMethodName()) {
+    return make_unique<GetPeersRpcMethod>();
+  }
 #endif // ENABLE_BITTORRENT
+
 #ifdef ENABLE_METALINK
-  }
-  else if(methodName == AddMetalinkRpcMethod::getMethodName()) {
+  if (methodName == AddMetalinkRpcMethod::getMethodName()) {
     return make_unique<AddMetalinkRpcMethod>();
-#endif // ENABLE_METALINK
   }
-  else if(methodName == RemoveRpcMethod::getMethodName()) {
+#endif // ENABLE_METALINK
+
+  if (methodName == RemoveRpcMethod::getMethodName()) {
     return make_unique<RemoveRpcMethod>();
-  } else if(methodName == PauseRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == PauseRpcMethod::getMethodName()) {
     return make_unique<PauseRpcMethod>();
-  } else if(methodName == ForcePauseRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ForcePauseRpcMethod::getMethodName()) {
     return make_unique<ForcePauseRpcMethod>();
-  } else if(methodName == PauseAllRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == PauseAllRpcMethod::getMethodName()) {
     return make_unique<PauseAllRpcMethod>();
-  } else if(methodName == ForcePauseAllRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ForcePauseAllRpcMethod::getMethodName()) {
     return make_unique<ForcePauseAllRpcMethod>();
-  } else if(methodName == UnpauseRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == UnpauseRpcMethod::getMethodName()) {
     return make_unique<UnpauseRpcMethod>();
-  } else if(methodName == UnpauseAllRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == UnpauseAllRpcMethod::getMethodName()) {
     return make_unique<UnpauseAllRpcMethod>();
-  } else if(methodName == ForceRemoveRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ForceRemoveRpcMethod::getMethodName()) {
     return make_unique<ForceRemoveRpcMethod>();
-  } else if(methodName == ChangePositionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ChangePositionRpcMethod::getMethodName()) {
     return make_unique<ChangePositionRpcMethod>();
-  } else if(methodName == TellStatusRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == TellStatusRpcMethod::getMethodName()) {
     return make_unique<TellStatusRpcMethod>();
-  } else if(methodName == GetUrisRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetUrisRpcMethod::getMethodName()) {
     return make_unique<GetUrisRpcMethod>();
-  } else if(methodName == GetFilesRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetFilesRpcMethod::getMethodName()) {
     return make_unique<GetFilesRpcMethod>();
-#ifdef ENABLE_BITTORRENT
   }
-  else if(methodName == GetPeersRpcMethod::getMethodName()) {
-    return make_unique<GetPeersRpcMethod>();
-#endif // ENABLE_BITTORRENT
-  } else if(methodName == GetServersRpcMethod::getMethodName()) {
+
+  if (methodName == GetServersRpcMethod::getMethodName()) {
     return make_unique<GetServersRpcMethod>();
-  } else if(methodName == TellActiveRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == TellActiveRpcMethod::getMethodName()) {
     return make_unique<TellActiveRpcMethod>();
-  } else if(methodName == TellWaitingRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == TellWaitingRpcMethod::getMethodName()) {
     return make_unique<TellWaitingRpcMethod>();
-  } else if(methodName == TellStoppedRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == TellStoppedRpcMethod::getMethodName()) {
     return make_unique<TellStoppedRpcMethod>();
-  } else if(methodName == GetOptionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetOptionRpcMethod::getMethodName()) {
     return make_unique<GetOptionRpcMethod>();
-  } else if(methodName == ChangeUriRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ChangeUriRpcMethod::getMethodName()) {
     return make_unique<ChangeUriRpcMethod>();
-  } else if(methodName == ChangeOptionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ChangeOptionRpcMethod::getMethodName()) {
     return make_unique<ChangeOptionRpcMethod>();
-  } else if(methodName == GetGlobalOptionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetGlobalOptionRpcMethod::getMethodName()) {
     return make_unique<GetGlobalOptionRpcMethod>();
-  } else if(methodName == ChangeGlobalOptionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ChangeGlobalOptionRpcMethod::getMethodName()) {
     return make_unique<ChangeGlobalOptionRpcMethod>();
-  } else if(methodName == PurgeDownloadResultRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == PurgeDownloadResultRpcMethod::getMethodName()) {
     return make_unique<PurgeDownloadResultRpcMethod>();
-  } else if(methodName == RemoveDownloadResultRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == RemoveDownloadResultRpcMethod::getMethodName()) {
     return make_unique<RemoveDownloadResultRpcMethod>();
-  } else if(methodName == GetVersionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetVersionRpcMethod::getMethodName()) {
     return make_unique<GetVersionRpcMethod>();
-  } else if(methodName == GetSessionInfoRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetSessionInfoRpcMethod::getMethodName()) {
     return make_unique<GetSessionInfoRpcMethod>();
-  } else if(methodName == ShutdownRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ShutdownRpcMethod::getMethodName()) {
     return make_unique<ShutdownRpcMethod>();
-  } else if(methodName == ForceShutdownRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == ForceShutdownRpcMethod::getMethodName()) {
     return make_unique<ForceShutdownRpcMethod>();
-  } else if(methodName == GetGlobalStatRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == GetGlobalStatRpcMethod::getMethodName()) {
     return make_unique<GetGlobalStatRpcMethod>();
-  } else if(methodName == SaveSessionRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == SaveSessionRpcMethod::getMethodName()) {
     return make_unique<SaveSessionRpcMethod>();
-  } else if(methodName == SystemMulticallRpcMethod::getMethodName()) {
+  }
+
+  if (methodName == SystemMulticallRpcMethod::getMethodName()) {
     return make_unique<SystemMulticallRpcMethod>();
-  } else {
-    return nullptr;
   }
+
+  return nullptr;
 }
 } // namespace
 
 RpcMethod* getMethod(const std::string& methodName)
 {
   auto itr = cache.find(methodName);
-  if(itr == cache.end()) {
+  if (itr == std::end(cache)) {
     auto m = createMethod(methodName);
-    if(m) {
+    if (m) {
       auto rv = cache.insert(std::make_pair(methodName, std::move(m)));
       return (*rv.first).second.get();
-    } else {
-      if(!noSuchRpcMethod) {
-        noSuchRpcMethod = make_unique<NoSuchMethodRpcMethod>();
-      }
-      return noSuchRpcMethod.get();
     }
-  } else {
-    return (*itr).second.get();
+
+    if (!noSuchRpcMethod) {
+      noSuchRpcMethod = make_unique<NoSuchMethodRpcMethod>();
+    }
+
+    return noSuchRpcMethod.get();
   }
+
+  return (*itr).second.get();
 }
 
 } // namespace rpc