Auto merge of #380 - est31:no_more_deprecation_warnings, r=jdm

No more deprecation warnings

CI is failing right now in my other PR #379 on beta and nightly because of deprecation warnings and a deny warnings directive. This PR fixes all deprecation warnings that rustc 1.37.0 beta is reporting.

Author: bors-servo

html5ever/src/tokenizer/char_ref/mod.rs

@@ -244,14 +244,14 @@ impl CharRefTokenizer {
 
         let (c, error) = match self.num {
             n if (n > 0x10FFFF) || self.num_too_big => ('\u{fffd}', true),
-            0x00 | 0xD800...0xDFFF => ('\u{fffd}', true),
+            0x00 | 0xD800..=0xDFFF => ('\u{fffd}', true),
 
-            0x80...0x9F => match data::C1_REPLACEMENTS[(self.num - 0x80) as usize] {
+            0x80..=0x9F => match data::C1_REPLACEMENTS[(self.num - 0x80) as usize] {
                 Some(c) => (c, true),
                 None => (conv(self.num), true),
             },
 
-            0x01...0x08 | 0x0B | 0x0D...0x1F | 0x7F | 0xFDD0...0xFDEF => (conv(self.num), true),
+            0x01..=0x08 | 0x0B | 0x0D..=0x1F | 0x7F | 0xFDD0..=0xFDEF => (conv(self.num), true),
 
             n if (n & 0xFFFE) == 0xFFFE => (conv(n), true),
 

html5ever/src/tokenizer/mod.rs

@@ -264,7 +264,7 @@ impl<Sink: TokenSink> Tokenizer<Sink> {
 
         if self.opts.exact_errors &&
             match c as u32 {
-                0x01...0x08 | 0x0B | 0x0E...0x1F | 0x7F...0x9F | 0xFDD0...0xFDEF => true,
+                0x01..=0x08 | 0x0B | 0x0E..=0x1F | 0x7F..=0x9F | 0xFDD0..=0xFDEF => true,
                 n if (n & 0xFFFE) == 0xFFFE => true,
                 _ => false,
             }

html5ever/src/util/str.rs

@@ -19,15 +19,15 @@ pub fn to_escaped_string<T: fmt::Debug>(x: &T) -> String {
 /// letter, otherwise None.
 pub fn lower_ascii_letter(c: char) -> Option<char> {
     match c {
-        'a'...'z' => Some(c),
-        'A'...'Z' => Some((c as u8 - b'A' + b'a') as char),
+        'a'..='z' => Some(c),
+        'A'..='Z' => Some((c as u8 - b'A' + b'a') as char),
         _ => None,
     }
 }
 
 /// Is the character an ASCII alphanumeric character?
 pub fn is_ascii_alnum(c: char) -> bool {
-    matches!(c, '0'...'9' | 'a'...'z' | 'A'...'Z')
+    matches!(c, '0'..='9' | 'a'..='z' | 'A'..='Z')
 }
 
 /// ASCII whitespace characters, as defined by

html5ever/tests/tree_builder.rs

@@ -188,15 +188,15 @@ fn make_test_desc_with_scripting_flag(
 ) -> TestDescAndFn {
     let get_field = |key| {
         let field = fields.get(key).expect("missing field");
-        field.trim_right_matches('\n').to_string()
+        field.trim_end_matches('\n').to_string()
     };
 
     let mut data = fields.get("data").expect("missing data").to_string();
     data.pop();
     let expected = get_field("document");
     let context = fields
         .get("document-fragment")
-        .map(|field| context_name(field.trim_right_matches('\n')));
+        .map(|field| context_name(field.trim_end_matches('\n')));
     let ignore = ignores.contains(name);
     let mut name = name.to_owned();
     if scripting_enabled {
@@ -293,7 +293,7 @@ fn main() {
         let f = fs::File::open(&src_dir.join("data/test/ignore")).unwrap();
         let r = io::BufReader::new(f);
         for ln in r.lines() {
-            ignores.insert(ln.unwrap().trim_right().to_string());
+            ignores.insert(ln.unwrap().trim_end().to_string());
         }
     }
 

xml5ever/src/tokenizer/char_ref/mod.rs

@@ -243,14 +243,14 @@ impl CharRefTokenizer {
 
         let (c, error) = match self.num {
             n if (n > 0x10FFFF) || self.num_too_big => ('\u{fffd}', true),
-            0x00 | 0xD800...0xDFFF => ('\u{fffd}', true),
+            0x00 | 0xD800..=0xDFFF => ('\u{fffd}', true),
 
-            0x80...0x9F => match data::C1_REPLACEMENTS[(self.num - 0x80) as usize] {
+            0x80..=0x9F => match data::C1_REPLACEMENTS[(self.num - 0x80) as usize] {
                 Some(c) => (c, true),
                 None => (conv(self.num), true),
             },
 
-            0x01...0x08 | 0x0B | 0x0D...0x1F | 0x7F | 0xFDD0...0xFDEF => (conv(self.num), true),
+            0x01..=0x08 | 0x0B | 0x0D..=0x1F | 0x7F | 0xFDD0..=0xFDEF => (conv(self.num), true),
 
             n if (n & 0xFFFE) == 0xFFFE => (conv(n), true),
 

xml5ever/src/tokenizer/mod.rs

@@ -247,7 +247,7 @@ impl<Sink: TokenSink> XmlTokenizer<Sink> {
         // Exclude forbidden Unicode characters
         if self.opts.exact_errors &&
             match c as u32 {
-                0x01...0x08 | 0x0B | 0x0E...0x1F | 0x7F...0x9F | 0xFDD0...0xFDEF => true,
+                0x01..=0x08 | 0x0B | 0x0E..=0x1F | 0x7F..=0x9F | 0xFDD0..=0xFDEF => true,
                 n if (n & 0xFFFE) == 0xFFFE => true,
                 _ => false,
             }

xml5ever/src/tree_builder/mod.rs

@@ -240,7 +240,7 @@ where
 
     /// Call the `Tracer`'s `trace_handle` method on every `Handle` in the tree builder's
     /// internal state.  This is intended to support garbage-collected DOMs.
-    pub fn trace_handles(&self, tracer: &Tracer<Handle = Handle>) {
+    pub fn trace_handles(&self, tracer: &dyn Tracer<Handle = Handle>) {
         tracer.trace_handle(&self.doc_handle);
         for e in self.open_elems.iter() {
             tracer.trace_handle(&e);

xml5ever/src/util/mod.rs

@@ -9,7 +9,7 @@
 
 /// Is the character an ASCII alphanumeric character?
 pub fn is_ascii_alnum(c: char) -> bool {
-    matches!(c, '0'...'9' | 'a'...'z' | 'A'...'Z')
+    matches!(c, '0'..='9' | 'a'..='z' | 'A'..='Z')
 }
 
 #[cfg(test)]

xml5ever/tests/tree_builder.rs

@@ -180,7 +180,7 @@ fn make_xml_test(
 ) {
     let get_field = |key| {
         let field = fields.get(key).expect("missing field");
-        field.trim_right_matches('\n').to_string()
+        field.trim_end_matches('\n').to_string()
     };
 
     let data = get_field("data");
@@ -249,7 +249,7 @@ fn run() {
     if let Ok(f) = fs::File::open(&src_dir.join("data/test/ignore")) {
         let r = io::BufReader::new(f);
         for ln in r.lines() {
-            ignores.insert(ln.unwrap().trim_right().to_string());
+            ignores.insert(ln.unwrap().trim_end().to_string());
         }
     }