Przeglądaj źródła

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
	src/common/crypto.c
Nick Mathewson 13 lat temu
rodzic
commit
b1b6552251
5 zmienionych plików z 16 dodań i 9 usunięć
  1. 6 0
      changes/doxygen
  2. 0 1
      src/common/crypto.c
  3. 3 3
      src/common/log.c
  4. 4 2
      src/test/test_crypto.c
  5. 3 3
      src/test/test_dir.c

+ 6 - 0
changes/doxygen

@@ -0,0 +1,6 @@
+  o Documentation changes
+    - Modernize the doxygen configuration file slightly.  Fixes bug 2707.
+    - Resolve all doxygen warnings except those for missing documentation.
+      Fixes bug 2705.
+    - Add doxygen documentation for more functions, fields, and types.
+

+ 0 - 1
src/common/crypto.c

@@ -2711,4 +2711,3 @@ setup_openssl_threading(void)
 #endif
 /** @} */
 
-

+ 3 - 3
src/common/log.c

@@ -409,7 +409,7 @@ logv(int severity, log_domain_mask_t domain, const char *funcname,
 
 /** Output a message to the log.  It gets logged to all logfiles that
  * care about messages with <b>severity</b> in <b>domain</b>. The content
- * if formatted printf style 
+ * is formatted printf style basedc on <b>format</b> and extra arguments.
  * */
 void
 tor_log(int severity, log_domain_mask_t domain, const char *format, ...)
@@ -425,8 +425,8 @@ tor_log(int severity, log_domain_mask_t domain, const char *format, ...)
 /** Output a message to the log, prefixed with a function name <b>fn</b>. */
 #ifdef __GNUC__
 /** GCC-based implementation of the log_fn backend, used when we have
- * variadic macros. All arguments are as for log_fn, except for <b>fn</b>, which
- * is the name of the calling functions. */
+ * variadic macros. All arguments are as for log_fn, except for
+ * <b>fn</b>, which is the name of the calling functions. */
 void
 _log_fn(int severity, log_domain_mask_t domain, const char *fn,
         const char *format, ...)

+ 4 - 2
src/test/test_crypto.c

@@ -384,12 +384,14 @@ test_crypto_pk(void)
   /* Now try signing. */
   strlcpy(data1, "Ossifrage", 1024);
   test_eq(128, crypto_pk_private_sign(pk1, data2, sizeof(data2), data1, 10));
-  test_eq(10, crypto_pk_public_checksig(pk1, data3, sizeof(data3), data2, 128));
+  test_eq(10,
+          crypto_pk_public_checksig(pk1, data3, sizeof(data3), data2, 128));
   test_streq(data3, "Ossifrage");
   /* Try signing digests. */
   test_eq(128, crypto_pk_private_sign_digest(pk1, data2, sizeof(data2),
                                              data1, 10));
-  test_eq(20, crypto_pk_public_checksig(pk1, data3, sizeof(data3), data2, 128));
+  test_eq(20,
+          crypto_pk_public_checksig(pk1, data3, sizeof(data3), data2, 128));
   test_eq(0, crypto_pk_public_checksig_digest(pk1, data1, 10, data2, 128));
   test_eq(-1, crypto_pk_public_checksig_digest(pk1, data1, 11, data2, 128));
 

+ 3 - 3
src/test/test_dir.c

@@ -748,11 +748,11 @@ test_dir_v3_networkstatus(void)
   sign_skey_leg1 = pk_generate(4);
 
   test_assert(!crypto_pk_read_private_key_from_string(sign_skey_1,
-                                                      AUTHORITY_SIGNKEY_1, -1));
+                                                   AUTHORITY_SIGNKEY_1, -1));
   test_assert(!crypto_pk_read_private_key_from_string(sign_skey_2,
-                                                      AUTHORITY_SIGNKEY_2, -1));
+                                                   AUTHORITY_SIGNKEY_2, -1));
   test_assert(!crypto_pk_read_private_key_from_string(sign_skey_3,
-                                                      AUTHORITY_SIGNKEY_3, -1));
+                                                   AUTHORITY_SIGNKEY_3, -1));
 
   test_assert(!crypto_pk_cmp_keys(sign_skey_1, cert1->signing_key));
   test_assert(!crypto_pk_cmp_keys(sign_skey_2, cert2->signing_key));