minor cleanup of CL_SendMove comments regarding packet timing

changed predicted move sending code to not care about the value of cls.signon

git-svn-id: svn://svn.icculus.org/twilight/trunk/darkplaces@6934 d7cf8633-e32d-0410-b094-e92efae38249

diff --git a/cl_input.c b/cl_input.c
index 720f8f097a234ffbc7f461e35b5e2d42b2b09c27..b3436b84d3da4887dbfe31f48e3c574f420c1197 100644 (file)
--- a/cl_input.c
+++ b/cl_input.c
@@ -1272,37 +1272,35 @@ void CL_SendMove(void)
        if (!cls.netcon)
                return;
 
+       // don't send too often or else network connections can get clogged by a high renderer framerate
        packettime = 1.0 / bound(10, cl_netinputpacketspersecond.value, 100);
-       // on quakeworld servers the server replies to client input, so we send
-       // packets whenever we want to
-       // on non-quakeworld servers the client replies to server updates
+       // quakeworld servers take only frametimes
+       // predicted dp7 servers take current interpolation time
+       // unpredicted servers take an echo of the latest server timestamp
        if (cls.protocol == PROTOCOL_QUAKEWORLD)
        {
-               // don't send too often or else network connections can get clogged by a high renderer framerate
                if (realtime < lastsendtime + packettime)
                        return;
                cl.cmd.time = realtime;
        }
-       else if (cl.movement_predicted || cls.signon < SIGNONS)
+       else if (cl.movement_predicted)
        {
-               // don't send too often or else network connections can get clogged by a high renderer framerate
                if (realtime < lastsendtime + packettime)
                        return;
-               cl.cmd.time = cls.protocol == PROTOCOL_QUAKEWORLD ? realtime : cl.time;
+               cl.cmd.time = cl.time;
        }
        else
        {
-               // if not predicted, we should just reply to server packets, and
-               // report the real latest packet time rather than our interpolated
-               // time
+               // unpredicted movement should be sent immediately whenever a server
+               // packet is received, to minimize ping times
                if (!cl.movement_needupdate && realtime < lastsendtime + packettime)
                        return;
-               cl.cmd.time = cls.protocol == PROTOCOL_QUAKEWORLD ? realtime : cl.mtime[0];
+               cl.cmd.time = cl.mtime[0];
        }
        // don't let it fall behind if CL_SendMove hasn't been called recently
        // (such is the case when framerate is too low for instance)
        lastsendtime = bound(realtime, lastsendtime + packettime, realtime + packettime);
-       // clear the note down that we sent a packet recently
+       // set the flag indicating that we sent a packet recently
        cl.movement_needupdate = false;