diff options
| author | Nick Mathewson <nickm@torproject.org> | 2011-08-30 20:42:51 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2011-08-30 20:44:42 -0400 |
| commit | 393e4fb5b5529c666438ef04ebd076f90dd287fa (patch) | |
| tree | a38b5288d5a6a8374d9e4b7e79d8f7745891aa8f /src/test/test.c | |
| parent | 561ab14a5ed64dd3994fe23073e74efa3498e589 (diff) | |
| download | tor-393e4fb5b5529c666438ef04ebd076f90dd287fa.tar.gz tor-393e4fb5b5529c666438ef04ebd076f90dd287fa.zip | |
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.
Diffstat (limited to 'src/test/test.c')
| -rw-r--r-- | src/test/test.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/test/test.c b/src/test/test.c index 9b24a99b58..b5b744eba7 100644 --- a/src/test/test.c +++ b/src/test/test.c @@ -455,7 +455,7 @@ test_circuit_timeout(void) timeout1 = circuit_build_times_calculate_timeout(&estimate, CBT_DEFAULT_QUANTILE_CUTOFF/100.0); circuit_build_times_set_timeout(&estimate); - log_notice(LD_CIRC, "Timeout1 is %lf, Xm is %d", timeout1, estimate.Xm); + log_notice(LD_CIRC, "Timeout1 is %f, Xm is %d", timeout1, estimate.Xm); /* 2% error */ } while (fabs(circuit_build_times_cdf(&initial, timeout0) - circuit_build_times_cdf(&initial, timeout1)) > 0.02); @@ -470,7 +470,7 @@ test_circuit_timeout(void) CBT_DEFAULT_QUANTILE_CUTOFF/100.0); circuit_build_times_set_timeout(&final); - log_notice(LD_CIRC, "Timeout2 is %lf, Xm is %d", timeout2, final.Xm); + log_notice(LD_CIRC, "Timeout2 is %f, Xm is %d", timeout2, final.Xm); /* 5% here because some accuracy is lost due to histogram conversion */ test_assert(fabs(circuit_build_times_cdf(&initial, timeout0) - |