fix darkplaces-dedicated.exe building to open a console

[xonotic/darkplaces.git] / host.c

diff --git a/host.c b/host.c
index 465545ad74c7f7ac088537e0d80f5c2ae1c1e202..c5c783b56eec9cd3dd702f078da35ad2f68a2e0e 100644 (file)
--- a/host.c
+++ b/host.c
@@ -425,26 +425,25 @@ void SV_DropClient (qboolean crash)
        int i;
        client_t *client;
 
+       Con_Printf ("Client \"%s\" dropped\n", host_client->name);
+
        if (!crash)
        {
                // send any final messages (don't check for errors)
-               if (NET_CanSendMessage (host_client->netconnection))
+               if (host_client->netconnection && !host_client->netconnection->disconnected)
                {
+#if 1
+                       // LordHavoc: no opportunity for resending, so reliable is silly
                        MSG_WriteByte (&host_client->message, svc_disconnect);
-                       NET_SendMessage (host_client->netconnection, &host_client->message);
-               }
-
-               if (sv.active && host_client->edict && host_client->spawned) // LordHavoc: don't call QC if server is dead (avoids recursive Host_Error in some mods when they run out of edicts)
-               {
-               // call the prog function for removing a client
-               // this will set the body to a dead frame, among other things
-                       saveSelf = pr_global_struct->self;
-                       pr_global_struct->self = EDICT_TO_PROG(host_client->edict);
-                       PR_ExecuteProgram (pr_global_struct->ClientDisconnect, "QC function ClientDisconnect is missing");
-                       pr_global_struct->self = saveSelf;
+                       NET_SendUnreliableMessage (host_client->netconnection, &host_client->message);
+#else
+                       if (NET_CanSendMessage (host_client->netconnection))
+                       {
+                               MSG_WriteByte (&host_client->message, svc_disconnect);
+                               NET_SendMessage (host_client->netconnection, &host_client->message);
+                       }
+#endif
                }
-
-               Sys_Printf ("Client %s removed\n",host_client->name);
        }
 
 // break the net connection
@@ -453,11 +452,24 @@ void SV_DropClient (qboolean crash)
 
 // free the client (the body stays around)
        host_client->active = false;
+       // note: don't clear name yet
+       net_activeconnections--;
+
+       if (sv.active && host_client->edict && host_client->spawned) // LordHavoc: don't call QC if server is dead (avoids recursive Host_Error in some mods when they run out of edicts)
+       {
+       // call the prog function for removing a client
+       // this will set the body to a dead frame, among other things
+               saveSelf = pr_global_struct->self;
+               pr_global_struct->self = EDICT_TO_PROG(host_client->edict);
+               PR_ExecuteProgram (pr_global_struct->ClientDisconnect, "QC function ClientDisconnect is missing");
+               pr_global_struct->self = saveSelf;
+       }
+
+       // now clear name (after ClientDisconnect was called)
        host_client->name[0] = 0;
        host_client->old_frags = -999999;
-       net_activeconnections--;
 
-// send notification to all clients
+       // send notification to all clients
        for (i=0, client = svs.clients ; i<svs.maxclients ; i++, client++)
        {
                if (!client->active)
@@ -541,7 +553,7 @@ void Host_ShutdownServer(qboolean crash)
 
        for (i=0, host_client = svs.clients ; i<svs.maxclients ; i++, host_client++)
                if (host_client->active)
-                       SV_DropClient(crash);
+                       SV_DropClient(crash); // server shutdown
 
 //
 // clear structures