[tor-commits] [tor/master] Fix a couple of harmless clang3.2 warnings

nickm at torproject.org nickm at torproject.org
Mon Dec 31 17:26:12 UTC 2012


commit 11e8a445c3051f017600b27999f0d666b7d63b04
Author: Sebastian Hahn <sebastian at torproject.org>
Date:   Mon Dec 31 18:23:28 2012 +0100

    Fix a couple of harmless clang3.2 warnings
---
 changes/clang_enum_warnings |    3 +++
 src/common/crypto.c         |    2 +-
 src/or/dirvote.c            |    4 ++--
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/changes/clang_enum_warnings b/changes/clang_enum_warnings
new file mode 100644
index 0000000..50de605
--- /dev/null
+++ b/changes/clang_enum_warnings
@@ -0,0 +1,3 @@
+  o Code simplifications and refactoring:
+    - Get rid of a couple of harmless clang warnings, where we compared
+      enums to ints. These warnings are newly introduced in clang 3.2.
diff --git a/src/common/crypto.c b/src/common/crypto.c
index 39f5a4a..4cf9a9c 100644
--- a/src/common/crypto.c
+++ b/src/common/crypto.c
@@ -1458,7 +1458,7 @@ crypto_digest256(char *digest, const char *m, size_t len,
 int
 crypto_digest_all(digests_t *ds_out, const char *m, size_t len)
 {
-  digest_algorithm_t i;
+  int i;
   tor_assert(ds_out);
   memset(ds_out, 0, sizeof(*ds_out));
   if (crypto_digest(ds_out->d[DIGEST_SHA1], m, len) < 0)
diff --git a/src/or/dirvote.c b/src/or/dirvote.c
index 1b9af0f..3000ee2 100644
--- a/src/or/dirvote.c
+++ b/src/or/dirvote.c
@@ -2223,7 +2223,7 @@ networkstatus_add_detached_signatures(networkstatus_t *target,
   {
     digests_t *digests = strmap_get(sigs->digests, flavor);
     int n_matches = 0;
-    digest_algorithm_t alg;
+    int alg;
     if (!digests) {
       *msg_out = "No digests for given consensus flavor";
       return -1;
@@ -3471,7 +3471,7 @@ dirvote_free_all(void)
 const char *
 dirvote_get_pending_consensus(consensus_flavor_t flav)
 {
-  tor_assert(((int)flav) >= 0 && flav < N_CONSENSUS_FLAVORS);
+  tor_assert(((int)flav) >= 0 && (int)flav < N_CONSENSUS_FLAVORS);
   return pending_consensuses[flav].body;
 }
 



More information about the tor-commits mailing list