]> git.ipfire.org Git - thirdparty/util-linux.git/commitdiff
hwclock: use fabs() instead abs() when argument is double
authorSami Kerola <kerolasa@iki.fi>
Sun, 28 Sep 2014 19:51:51 +0000 (20:51 +0100)
committerKarel Zak <kzak@redhat.com>
Wed, 1 Oct 2014 11:09:36 +0000 (13:09 +0200)
sys-utils/hwclock.c:1219:13: warning: using integer absolute value function
      'abs' when argument is of floating point type [-Wabsolute-value]

Signed-off-by: Sami Kerola <kerolasa@iki.fi>
sys-utils/hwclock.c

index 493ee8808f853383577b7e6dfeaf15cb96a4aeb0..e7801583b82f021f814a3a9eb5433fb81404aa03 100644 (file)
@@ -60,6 +60,7 @@
 #include <errno.h>
 #include <getopt.h>
 #include <limits.h>
+#include <math.h>
 #include <stdarg.h>
 #include <stdio.h>
 #include <stdlib.h>
@@ -1042,7 +1043,7 @@ adjust_drift_factor(struct adjtime *adjtime_p,
                /* New drift factor */
                drift_factor = adjtime_p->drift_factor + factor_adjust;
 
-               if (abs(drift_factor) > MAX_DRIFT) {
+               if (fabs(drift_factor) > MAX_DRIFT) {
                        if (debug)
                                printf(_("Clock drift factor was calculated as "
                                         "%f seconds/day.\n"
@@ -1215,7 +1216,7 @@ do_adjustment(struct adjtime *adjtime_p,
                if (debug)
                        printf(_("Not setting clock because last adjustment time is zero, "
                                 "so history is bad.\n"));
-       } else if (abs(adjtime_p->drift_factor) > MAX_DRIFT) {
+       } else if (fabs(adjtime_p->drift_factor) > MAX_DRIFT) {
                if (debug)
                        printf(_("Not setting clock because drift factor %f is far too high.\n"),
                                adjtime_p->drift_factor);