For integers, if !(E<G), then we can infer that E>=G.

This means that "if (E<G) {abc} else if (E>=G) {def}" can be replaced with
"if (E<G) {abc} else {def}"

Doing the second test explicitly made my mingw gcc nervous that we might
never be initializing casename.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/or/dirvote.c b/src/or/dirvote.c
index 505af38cfc..706e6e4480 100644
--- a/src/or/dirvote.c
+++ b/src/or/dirvote.c
@@ -842,7 +842,7 @@ networkstatus_compute_bw_weights_v9(smartlist_t *chunks, int64_t G, int64_t M,
         casename = "Case 2a (E scarce)";
         Wed = weight_scale;
         Wgd = 0;
-      } else if (E >= G) {
+      } else { /* E >= G */
         casename = "Case 2a (G scarce)";
         Wed = 0;
         Wgd = weight_scale;