Fix a silly mistake in the tor_mathlog() documentation. Give it a unit test.

diff --git a/src/common/util.c b/src/common/util.c
index 071ba8f63531327514fdcba8a417c1783631d411..7d72a896f983888c17ccf9c0cf9ed404de9381af 100644 (file)
--- a/src/common/util.c
+++ b/src/common/util.c
@@ -322,7 +322,7 @@ tor_log_mallinfo(int severity)
  * ===== */
 
 /**
- * Returns the natural logarithm of d base 2.  We defined this wrapper here so
+ * Returns the natural logarithm of d base e.  We defined this wrapper here so
  * to avoid conflicts with old versions of tor_log(), which were named log().
  */
 double

diff --git a/src/test/test_util.c b/src/test/test_util.c
index ad214b9e677db8a03b725eb1c09ff384a6739fc9..bed33fac2504422a6d9bc7fe9f7c2d721d0a2757 100644 (file)
--- a/src/test/test_util.c
+++ b/src/test/test_util.c
@@ -17,6 +17,7 @@
 #ifdef _WIN32
 #include <tchar.h>
 #endif
+#include <math.h>
 
 /* XXXX this is a minimal wrapper to make the unit tests compile with the
  * changed tor_timegm interface. */
@@ -3240,6 +3241,20 @@ test_util_set_env_var_in_sl(void *ptr)
   smartlist_free(expected_resulting_env_vars);
 }
 
+static void
+test_util_mathlog(void *arg)
+{
+  double d;
+  (void) arg;
+
+  d = tor_mathlog(2.718281828);
+  tt_double_op(fabs(d - 1.0), <, .000001);
+  d = tor_mathlog(10);
+  tt_double_op(fabs(d - 2.30258509), <, .000001);
+ done:
+  ;
+}
+
 #define UTIL_LEGACY(name)                                               \
   { #name, legacy_test_helper, 0, &legacy_setup, test_util_ ## name }
 
@@ -3296,6 +3311,7 @@ struct testcase_t util_tests[] = {
   UTIL_TEST(read_file_eof_tiny_limit, 0),
   UTIL_TEST(read_file_eof_two_loops, 0),
   UTIL_TEST(read_file_eof_zero_bytes, 0),
+  UTIL_TEST(mathlog, 0),
   END_OF_TESTCASES
 };