(increase_mob): Call time() just once, the second call can only

introduce inaccuracy.  Simplify the loop; it still doesn't make much
sense.

diff --git a/src/lib/update/mobility.c b/src/lib/update/mobility.c
index 62c52605f329402aa5f4c39855626f7779a7594e..360a24c55d296c97b0075d6a8827030030903f56 100644 (file)
--- a/src/lib/update/mobility.c
+++ b/src/lib/update/mobility.c
@@ -79,8 +79,8 @@ increase_mob(time_t *counter, float mult)
        return 0;
     left = (secs % s_p_etu);
     do {
-       newmob = (float)(newetus * mult);
-       inewmob = (int)(newetus * mult);
+       newmob = newetus * mult;
+       inewmob = (int)newmob;
        if (newmob == inewmob || newetus > 7)
            break;
        newetus--;
@@ -89,8 +89,7 @@ increase_mob(time_t *counter, float mult)
     if (newetus <= 0)
        return 0;
 
-    time(counter);
-    *counter = *counter - left;
+    *counter = now - left;
 
     if (updating_mob)
        return newetus;