From 89fffcee883763cff9d8d44605a54db1ec22feed Mon Sep 17 00:00:00 2001 From: brain Date: Thu, 17 Jan 2008 16:40:17 +0000 Subject: The outer if (socket > -1) check in AddClient is obsolete and this is already checked in socket.cpp. We can remove and just have the inner check. git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@8730 e03df62e-2008-0410-955e-edbf42e46eb7 --- src/usermanager.cpp | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) (limited to 'src') diff --git a/src/usermanager.cpp b/src/usermanager.cpp index 193262681..1917c780e 100644 --- a/src/usermanager.cpp +++ b/src/usermanager.cpp @@ -155,14 +155,11 @@ void UserManager::AddClient(InspIRCd* Instance, int socket, int port, bool iscac } } - if (socket > -1) - { - if (!Instance->SE->AddFd(New)) - { - Instance->Log(DEBUG,"Internal error on new connection"); - User::QuitUser(Instance, New, "Internal error handling connection"); - } - } + if (!Instance->SE->AddFd(New)) + { + Instance->Log(DEBUG,"Internal error on new connection"); + User::QuitUser(Instance, New, "Internal error handling connection"); + } /* NOTE: even if dns lookups are *off*, we still need to display this. * BOPM and other stuff requires it. @@ -270,8 +267,8 @@ unsigned int UserManager::UnregisteredUserCount() /* return how many local registered users there are */ unsigned int UserManager::LocalUserCount() { - /* Doesnt count unregistered clients */ - return (ServerInstance->local_users.size() - this->UnregisteredUserCount()); + /* Doesnt count unregistered clients */ + return (ServerInstance->local_users.size() - this->UnregisteredUserCount()); } -- cgit v1.2.3