New system for client exits using CullList seems stable, needs testing

git-svn-id: http://svn.inspircd.org/repository/trunk/inspircd@2681 e03df62e-2008-0410-955e-edbf42e46eb7

4 files changed, 11 insertions, 17 deletions

diff --git a/src/cmd_nick.cpp b/src/cmd_nick.cpp
index fe600dc30..7b4ea1e98 100644
--- a/src/cmd_nick.cpp
+++ b/src/cmd_nick.cpp
@@ -49,7 +49,6 @@ using namespace std;
 #include "typedefs.h"
 #include "command_parse.h"
 #include "cmd_nick.h"
-#include "cull_list.h"
 
 extern ServerConfig* Config;
 extern InspIRCd* ServerInstance;
@@ -64,8 +63,6 @@ extern std::vector<userrec*> all_opers;
 extern std::vector<userrec*> local_users;
 extern userrec* fd_ref_table[65536];
 
-extern CullList* GlobalGoners;
-
 void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 {
 	char oldnick[NICKMAX];
@@ -194,7 +191,7 @@ void cmd_nick::Handle (char **parameters, int pcnt, userrec *user)
 	{
 		/* user is registered now, bit 0 = USER command, bit 1 = sent a NICK command */
 		FOREACH_MOD(I_OnUserRegister,OnUserRegister(user));
-		ConnectUser(user,GlobalGoners);
+		//ConnectUser(user,NULL);
 	}
 	if (user->registered == 7)
 	{
diff --git a/src/cmd_user.cpp b/src/cmd_user.cpp
index 7aa64b092..3d1754af3 100644
--- a/src/cmd_user.cpp
+++ b/src/cmd_user.cpp
@@ -48,7 +48,6 @@ using namespace std;
 #include "typedefs.h"
 #include "command_parse.h"
 #include "cmd_user.h"
-#include "cull_list.h"
 
 extern ServerConfig* Config;
 extern InspIRCd* ServerInstance;
@@ -63,8 +62,6 @@ extern std::vector<userrec*> all_opers;
 extern std::vector<userrec*> local_users;
 extern userrec* fd_ref_table[65536];
 
-extern CullList* GlobalGoners;
-
 void cmd_user::Handle (char **parameters, int pcnt, userrec *user)
 {
 	log(DEBUG,"Handling USER from cmd_user class!");
@@ -95,7 +92,7 @@ void cmd_user::Handle (char **parameters, int pcnt, userrec *user)
 	{
 		/* user is registered now, bit 0 = USER command, bit 1 = sent a NICK command */
 		FOREACH_MOD(I_OnUserRegister,OnUserRegister(user));
-		ConnectUser(user,GlobalGoners);
+		//ConnectUser(user,NULL);
 	}
 }
 	
diff --git a/src/userprocess.cpp b/src/userprocess.cpp
index 2398ff8fe..1acbb6547 100644
--- a/src/userprocess.cpp
+++ b/src/userprocess.cpp
@@ -283,11 +283,9 @@ bool DoBackgroundUserStuff(time_t TIME)
 
                 if ((curr) && (curr->fd != 0)) /* XXX - why are we checking fd != 0? --w00t */
                 {
-                        int currfd = curr->fd;
                         // we don't check the state of remote users.
 			if (IS_LOCAL(curr))
                         {
-                                curr->FlushWriteBuf();
                                 if (curr->GetWriteError() != "")
                                 {
                                         log(DEBUG,"InspIRCd: write error: %s",curr->GetWriteError().c_str());
@@ -329,6 +327,7 @@ bool DoBackgroundUserStuff(time_t TIME)
                                        curr->lastping = 0;
                                        curr->nping = TIME+curr->pingmax;       // was hard coded to 120
                                 }
+				curr->FlushWriteBuf();
                         }
                 }
         }
diff --git a/src/users.cpp b/src/users.cpp
index bde0f99fd..4db6dedc5 100644
--- a/src/users.cpp
+++ b/src/users.cpp
@@ -35,6 +35,7 @@ using namespace std;
 #include "message.h"
 #include "wildcard.h"
 #include "xline.h"
+#include "cull_list.h"
 
 extern InspIRCd* ServerInstance;
 extern int WHOWAS_STALE;
@@ -717,14 +718,14 @@ void FullConnectUser(userrec* user, CullList* Goners)
 
 
 /* shows the message of the day, and any other on-logon stuff */
-void ConnectUser(userrec *user, CullList* Goners)
-{
+//void ConnectUser(userrec *user)
+//{
         // dns is already done, things are fast. no need to wait for dns to complete just pass them straight on
-        if ((user->dns_done) && (user->registered >= 3) && (AllModulesReportReady(user)))
-        {
-                FullConnectUser(user, Goners);
-        }
-}
+        //if ((user->dns_done) && (user->registered >= 3) && (AllModulesReportReady(user)))
+        //{
+        //        FullConnectUser(user, Goners);
+        //}
+//}
 
 /* re-allocates a nick in the user_hash after they change nicknames,
  * returns a pointer to the new user as it may have moved */