Fix up the compare chain in nstime_delta().
authorGuy Harris <guy@alum.mit.edu>
Mon, 9 May 2016 23:51:29 +0000 (16:51 -0700)
committerGuy Harris <guy@alum.mit.edu>
Mon, 9 May 2016 23:52:33 +0000 (23:52 +0000)
The first case handles the two time stamps having the same seconds
value, so, in the subsequent cases, they're guaranteed not to have the
same seconds value; check for b->secs < a->secs, not for b->secs <= a->secs
(the two tests will always get the same value, as b->secs != a->secs),
to make it clearer what's being done.

Change-Id: I6d3806237dae0ea12af92ea0344a31a2c5322b12
Reviewed-on: https://code.wireshark.org/review/15325
Reviewed-by: Guy Harris <guy@alum.mit.edu>
wsutil/nstime.c

index e7ef252dda74762e08924733db5f9a1e0340e322..e4b2cab53e5626a246436467fc13b04f22060615 100644 (file)
@@ -96,7 +96,7 @@ void nstime_delta(nstime_t *delta, const nstime_t *b, const nstime_t *a )
            can never result. */
         delta->secs = 0;
         delta->nsecs = b->nsecs - a->nsecs;
-    } else if (b->secs <= a->secs) {
+    } else if (b->secs < a->secs) {
         /* The seconds part of b is less than the seconds part of a, so b is
            before a.