Wireshark-commits: [Wireshark-commits] master ea1ba62: Fix up the compare chain in nstime_delta().
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 9 May 2016 23:52:49 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ea1ba62aecc7116ab5764d5f0f1c33d4333a3bdf
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

ea1ba62 by Guy Harris (guy@xxxxxxxxxxxx):

    Fix up the compare chain in nstime_delta().
    
    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@xxxxxxxxxxxx>
    

Actions performed:

    from  9a8a454   CID 1215245, 1215247: NULL check for the param of proto_get_id()
    adds  ea1ba62   Fix up the compare chain in nstime_delta().


Summary of changes:
 wsutil/nstime.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)