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.
This commit is contained in:
Nick Mathewson 2010-02-25 17:00:14 -05:00
parent f0b5f87eab
commit 512cebadad

View File

@ -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;