Use %f with printf-style formatting, not %lf

For printf, %f and %lf are synonymous, since floats are promoted to
doubles when passed as varargs.  It's only for scanf that we need to
say "%lf" for doubles and "%f" for floats.

Apparenly, some older compilers think it's naughty to say %lf and like
to spew warnings about it.

Found by grarpamp.

1 files changed, 41 insertions, 41 deletions

diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index 42dbcacb5..aed4fd131 100644
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -2419,40 +2419,40 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
   // We use > 1 as the check for these because they are computed as integers.
   // Sometimes there are rounding errors.
   if (fabs(Wmm - weight_scale) > 1) {
-    log_warn(LD_BUG, "Wmm=%lf != "I64_FORMAT,
+    log_warn(LD_BUG, "Wmm=%f != "I64_FORMAT,
              Wmm, I64_PRINTF_ARG(weight_scale));
     valid = 0;
   }
 
   if (fabs(Wem - Wee) > 1) {
-    log_warn(LD_BUG, "Wem=%lf != Wee=%lf", Wem, Wee);
+    log_warn(LD_BUG, "Wem=%f != Wee=%f", Wem, Wee);
     valid = 0;
   }
 
   if (fabs(Wgm - Wgg) > 1) {
-    log_warn(LD_BUG, "Wgm=%lf != Wgg=%lf", Wgm, Wgg);
+    log_warn(LD_BUG, "Wgm=%f != Wgg=%f", Wgm, Wgg);
     valid = 0;
   }
 
   if (fabs(Weg - Wed) > 1) {
-    log_warn(LD_BUG, "Wed=%lf != Weg=%lf", Wed, Weg);
+    log_warn(LD_BUG, "Wed=%f != Weg=%f", Wed, Weg);
     valid = 0;
   }
 
   if (fabs(Wgg + Wmg - weight_scale) > 0.001*weight_scale) {
-    log_warn(LD_BUG, "Wgg=%lf != "I64_FORMAT" - Wmg=%lf", Wgg,
+    log_warn(LD_BUG, "Wgg=%f != "I64_FORMAT" - Wmg=%f", Wgg,
              I64_PRINTF_ARG(weight_scale), Wmg);
     valid = 0;
   }
 
   if (fabs(Wee + Wme - weight_scale) > 0.001*weight_scale) {
-    log_warn(LD_BUG, "Wee=%lf != "I64_FORMAT" - Wme=%lf", Wee,
+    log_warn(LD_BUG, "Wee=%f != "I64_FORMAT" - Wme=%f", Wee,
              I64_PRINTF_ARG(weight_scale), Wme);
     valid = 0;
   }
 
   if (fabs(Wgd + Wmd + Wed - weight_scale) > 0.001*weight_scale) {
-    log_warn(LD_BUG, "Wgd=%lf + Wmd=%lf + Wed=%lf != "I64_FORMAT,
+    log_warn(LD_BUG, "Wgd=%f + Wmd=%f + Wed=%f != "I64_FORMAT,
              Wgd, Wmd, Wed, I64_PRINTF_ARG(weight_scale));
     valid = 0;
   }
@@ -2507,10 +2507,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
     casename = "Case 1";
     if (fabs(Etotal-Mtotal) > 0.01*MAX(Etotal,Mtotal)) {
       log_warn(LD_DIR,
-               "Bw Weight Failure for %s: Etotal %lf != Mtotal %lf. "
+               "Bw Weight Failure for %s: Etotal %f != Mtotal %f. "
                "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                " T="I64_FORMAT". "
-               "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+               "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                casename, Etotal, Mtotal,
                I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2519,10 +2519,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
     }
     if (fabs(Etotal-Gtotal) > 0.01*MAX(Etotal,Gtotal)) {
       log_warn(LD_DIR,
-               "Bw Weight Failure for %s: Etotal %lf != Gtotal %lf. "
+               "Bw Weight Failure for %s: Etotal %f != Gtotal %f. "
                "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                " T="I64_FORMAT". "
-               "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+               "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                casename, Etotal, Gtotal,
                I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2531,10 +2531,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
     }
     if (fabs(Gtotal-Mtotal) > 0.01*MAX(Gtotal,Mtotal)) {
       log_warn(LD_DIR,
-               "Bw Weight Failure for %s: Mtotal %lf != Gtotal %lf. "
+               "Bw Weight Failure for %s: Mtotal %f != Gtotal %f. "
                "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                " T="I64_FORMAT". "
-               "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+               "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                casename, Mtotal, Gtotal,
                I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2563,10 +2563,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       // Rtotal < Stotal
       if (Rtotal > Stotal) {
         log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: Rtotal %lf > Stotal %lf. "
+                   "Bw Weight Failure for %s: Rtotal %f > Stotal %f. "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Rtotal, Stotal,
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2576,10 +2576,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       // Rtotal < T/3
       if (3*Rtotal > T) {
         log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: 3*Rtotal %lf > T "
+                   "Bw Weight Failure for %s: 3*Rtotal %f > T "
                    I64_FORMAT". G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT
                    " D="I64_FORMAT" T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Rtotal*3, I64_PRINTF_ARG(T),
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2589,10 +2589,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       // Stotal < T/3
       if (3*Stotal > T) {
         log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: 3*Stotal %lf > T "
+                   "Bw Weight Failure for %s: 3*Stotal %f > T "
                    I64_FORMAT". G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT
                    " D="I64_FORMAT" T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Stotal*3, I64_PRINTF_ARG(T),
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2602,11 +2602,11 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       // Mtotal > T/3
       if (3*Mtotal < T) {
         log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: 3*Mtotal %lf < T "
+                   "Bw Weight Failure for %s: 3*Mtotal %f < T "
                    I64_FORMAT". "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Mtotal*3, I64_PRINTF_ARG(T),
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2621,10 +2621,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
         casename = "Case 2b (balanced)";
         if (fabs(Etotal-Mtotal) > 0.01*MAX(Etotal,Mtotal)) {
           log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: Etotal %lf != Mtotal %lf. "
+                   "Bw Weight Failure for %s: Etotal %f != Mtotal %f. "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Etotal, Mtotal,
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2633,10 +2633,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
         }
         if (fabs(Etotal-Gtotal) > 0.01*MAX(Etotal,Gtotal)) {
           log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: Etotal %lf != Gtotal %lf. "
+                   "Bw Weight Failure for %s: Etotal %f != Gtotal %f. "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Etotal, Gtotal,
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2645,10 +2645,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
         }
         if (fabs(Gtotal-Mtotal) > 0.01*MAX(Gtotal,Mtotal)) {
           log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: Mtotal %lf != Gtotal %lf. "
+                   "Bw Weight Failure for %s: Mtotal %f != Gtotal %f. "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Mtotal, Gtotal,
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2658,10 +2658,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       } else {
         if (fabs(Etotal-Gtotal) > 0.01*MAX(Etotal,Gtotal)) {
           log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: Etotal %lf != Gtotal %lf. "
+                   "Bw Weight Failure for %s: Etotal %f != Gtotal %f. "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Etotal, Gtotal,
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2688,10 +2688,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       // Stotal < T/3
       if (3*Stotal > T) {
         log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: 3*Stotal %lf > T "
+                   "Bw Weight Failure for %s: 3*Stotal %f > T "
                    I64_FORMAT". G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT
                    " D="I64_FORMAT" T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, Stotal*3, I64_PRINTF_ARG(T),
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2701,10 +2701,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       if (NS >= M) {
         if (fabs(NStotal-Mtotal) > 0.01*MAX(NStotal,Mtotal)) {
           log_warn(LD_DIR,
-                   "Bw Weight Failure for %s: NStotal %lf != Mtotal %lf. "
+                   "Bw Weight Failure for %s: NStotal %f != Mtotal %f. "
                    "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                    " T="I64_FORMAT". "
-                   "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                   "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                    casename, NStotal, Mtotal,
                    I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                    I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2715,10 +2715,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
         // if NS < M, NStotal > T/3 because only one of G or E is scarce
         if (3*NStotal < T) {
           log_warn(LD_DIR,
-                     "Bw Weight Failure for %s: 3*NStotal %lf < T "
+                     "Bw Weight Failure for %s: 3*NStotal %f < T "
                      I64_FORMAT". G="I64_FORMAT" M="I64_FORMAT
                      " E="I64_FORMAT" D="I64_FORMAT" T="I64_FORMAT". "
-                     "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                     "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                      casename, NStotal*3, I64_PRINTF_ARG(T),
                      I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                      I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2730,10 +2730,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       casename = "Case 3b";
       if (fabs(Etotal-Mtotal) > 0.01*MAX(Etotal,Mtotal)) {
         log_warn(LD_DIR,
-                 "Bw Weight Failure for %s: Etotal %lf != Mtotal %lf. "
+                 "Bw Weight Failure for %s: Etotal %f != Mtotal %f. "
                  "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                  " T="I64_FORMAT". "
-                 "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                 "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                  casename, Etotal, Mtotal,
                  I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                  I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2742,10 +2742,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       }
       if (fabs(Etotal-Gtotal) > 0.01*MAX(Etotal,Gtotal)) {
         log_warn(LD_DIR,
-                 "Bw Weight Failure for %s: Etotal %lf != Gtotal %lf. "
+                 "Bw Weight Failure for %s: Etotal %f != Gtotal %f. "
                  "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                  " T="I64_FORMAT". "
-                 "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                 "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                  casename, Etotal, Gtotal,
                  I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                  I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),
@@ -2754,10 +2754,10 @@ networkstatus_verify_bw_weights(networkstatus_t *ns)
       }
       if (fabs(Gtotal-Mtotal) > 0.01*MAX(Gtotal,Mtotal)) {
         log_warn(LD_DIR,
-                 "Bw Weight Failure for %s: Mtotal %lf != Gtotal %lf. "
+                 "Bw Weight Failure for %s: Mtotal %f != Gtotal %f. "
                  "G="I64_FORMAT" M="I64_FORMAT" E="I64_FORMAT" D="I64_FORMAT
                  " T="I64_FORMAT". "
-                 "Wgg=%lf Wgd=%lf Wmg=%lf Wme=%lf Wmd=%lf Wee=%lf Wed=%lf",
+                 "Wgg=%f Wgd=%f Wmg=%f Wme=%f Wmd=%f Wee=%f Wed=%f",
                  casename, Mtotal, Gtotal,
                  I64_PRINTF_ARG(G), I64_PRINTF_ARG(M), I64_PRINTF_ARG(E),
                  I64_PRINTF_ARG(D), I64_PRINTF_ARG(T),