tests/opr/time-t.c: Use labs instead of abs for long argument
authorAnders Kaseorg <andersk@mit.edu>
Sat, 1 Aug 2015 09:54:42 +0000 (05:54 -0400)
committerBenjamin Kaduk <kaduk@mit.edu>
Sun, 2 Aug 2015 03:05:26 +0000 (23:05 -0400)
Resolves this warning with clang:

time-t.c:46:8: warning: absolute value function 'abs' given an argument of type 'long' but has parameter of type 'int' which may cause
      truncation of value [-Wabsolute-value]
    ok(abs(osTime - osNow) < 2, "opr_time_Now returns a reasonable value");
       ^
time-t.c:46:8: note: use function 'labs' instead
    ok(abs(osTime - osNow) < 2, "opr_time_Now returns a reasonable value");
       ^~~
       labs

Change-Id: Ib98069e1349161d936c8ada0e69f9b33d2f71ce3
Reviewed-on: http://gerrit.openafs.org/11965
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>

tests/opr/time-t.c

index d650c8f..5d91b65 100644 (file)
@@ -43,7 +43,7 @@ main(int argc, char **argv)
     is_int(0, opr_time_Now(&oprTime), "opr_time_Now succeeds");
     osNow = time(NULL);
     osTime = opr_time_ToSecs(&oprTime);
-    ok(abs(osTime - osNow) < 2, "opr_time_Now returns a reasonable value");
+    ok(labs(osTime - osNow) < 2, "opr_time_Now returns a reasonable value");
 
     return 0;
 }