Auto merge of #364 - froydnj:standard-binary-search, r=SimonSapin

idna: use standard binary_search_by

Doing this is arguably nicer and avoids an edge case when searching for
the last element in the table.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/364)
<!-- Reviewable:end -->

Author: bors-servo

idna/src/uts46.rs

@@ -12,6 +12,7 @@
 use self::Mapping::*;
 use punycode;
 use std::ascii::AsciiExt;
+use std::cmp::Ordering::{Equal, Less, Greater};
 use unicode_normalization::UnicodeNormalization;
 use unicode_normalization::char::is_combining_mark;
 use unicode_bidi::{BidiClass, bidi_class};
@@ -49,20 +50,16 @@ struct Range {
 }
 
 fn find_char(codepoint: char) -> &'static Mapping {
-    let mut min = 0;
-    let mut max = TABLE.len() - 1;
-    while max > min {
-        let mid = (min + max) >> 1;
-        if codepoint > TABLE[mid].to {
-           min = mid;
-        } else if codepoint < TABLE[mid].from {
-            max = mid;
+    let r = TABLE.binary_search_by(|ref range| {
+        if codepoint > range.to {
+            Less
+        } else if codepoint < range.from {
+            Greater
         } else {
-            min = mid;
-            max = mid;
+            Equal
         }
-    }
-    &TABLE[min].mapping
+    });
+    r.ok().map(|i| &TABLE[i].mapping).unwrap()
 }
 
 fn map_char(codepoint: char, flags: Flags, output: &mut String, errors: &mut Vec<Error>) {