Rust 2018 transition

saphyr/src/emitter.rs

@@ -1,7 +1,7 @@
 use std::convert::From;
 use std::error::Error;
 use std::fmt::{self, Display};
-use yaml::{Hash, Yaml};
+use crate::yaml::{Hash, Yaml};
 
 #[derive(Copy, Clone, Debug)]
 pub enum EmitError {
@@ -17,7 +17,7 @@ impl Error for EmitError {
         }
     }
 
-    fn cause(&self) -> Option<&Error> {
+    fn cause(&self) -> Option<&dyn Error> {
         None
     }
 }
@@ -38,7 +38,7 @@ impl From<fmt::Error> for EmitError {
 }
 
 pub struct YamlEmitter<'a> {
-    writer: &'a mut fmt::Write,
+    writer: &'a mut dyn fmt::Write,
     best_indent: usize,
     compact: bool,
 
@@ -48,7 +48,7 @@ pub struct YamlEmitter<'a> {
 pub type EmitResult = Result<(), EmitError>;
 
 // from serialize::json
-fn escape_str(wr: &mut fmt::Write, v: &str) -> Result<(), fmt::Error> {
+fn escape_str(wr: &mut dyn fmt::Write, v: &str) -> Result<(), fmt::Error> {
     wr.write_str("\"")?;
 
     let mut start = 0;
@@ -111,7 +111,7 @@ fn escape_str(wr: &mut fmt::Write, v: &str) -> Result<(), fmt::Error> {
 }
 
 impl<'a> YamlEmitter<'a> {
-    pub fn new(writer: &'a mut fmt::Write) -> YamlEmitter {
+    pub fn new(writer: &'a mut dyn fmt::Write) -> YamlEmitter {
         YamlEmitter {
             writer,
             best_indent: 2,
@@ -316,12 +316,12 @@ fn need_quotes(string: &str) -> bool {
             | '\"'
             | '\''
             | '\\'
-            | '\0'...'\x06'
+            | '\0'..='\x06'
             | '\t'
             | '\n'
             | '\r'
-            | '\x0e'...'\x1a'
+            | '\x0e'..='\x1a'
-            | '\x1c'...'\x1f' => true,
+            | '\x1c'..='\x1f' => true,
             _ => false,
         })
         || [
@@ -344,7 +344,7 @@ fn need_quotes(string: &str) -> bool {
 #[cfg(test)]
 mod test {
     use super::*;
-    use YamlLoader;
+    use crate::YamlLoader;
 
     #[test]
     fn test_emit_simple() {

saphyr/src/lib.rs

@@ -52,10 +52,10 @@ pub mod scanner;
 pub mod yaml;
 
 // reexport key APIs
-pub use emitter::{EmitError, YamlEmitter};
+pub use crate::emitter::{EmitError, YamlEmitter};
-pub use parser::Event;
+pub use crate::parser::Event;
-pub use scanner::ScanError;
+pub use crate::scanner::ScanError;
-pub use yaml::{Yaml, YamlLoader};
+pub use crate::yaml::{Yaml, YamlLoader};
 
 #[cfg(test)]
 mod tests {

saphyr/src/parser.rs

@@ -1,4 +1,4 @@
-use scanner::*;
+use crate::scanner::*;
 use std::collections::HashMap;
 
 #[derive(Clone, Copy, PartialEq, Debug, Eq)]

saphyr/src/scanner.rs

@@ -67,7 +67,7 @@ impl Error for ScanError {
         self.info.as_ref()
     }
 
-    fn cause(&self) -> Option<&Error> {
+    fn cause(&self) -> Option<&dyn Error> {
         None
     }
 }
@@ -199,7 +199,7 @@ fn is_digit(c: char) -> bool {
 #[inline]
 fn is_alpha(c: char) -> bool {
     match c {
-        '0'...'9' | 'a'...'z' | 'A'...'Z' => true,
+        '0'..='9' | 'a'..='z' | 'A'..='Z' => true,
         '_' | '-' => true,
         _ => false,
     }
@@ -211,9 +211,9 @@ fn is_hex(c: char) -> bool {
 #[inline]
 fn as_hex(c: char) -> u32 {
     match c {
-        '0'...'9' => (c as u32) - ('0' as u32),
+        '0'..='9' => (c as u32) - ('0' as u32),
-        'a'...'f' => (c as u32) - ('a' as u32) + 10,
+        'a'..='f' => (c as u32) - ('a' as u32) + 10,
-        'A'...'F' => (c as u32) - ('A' as u32) + 10,
+        'A'..='F' => (c as u32) - ('A' as u32) + 10,
         _ => unreachable!(),
     }
 }

saphyr/src/yaml.rs

@@ -1,6 +1,6 @@
 use linked_hash_map::LinkedHashMap;
-use parser::*;
+use crate::parser::*;
-use scanner::{Marker, ScanError, TScalarStyle, TokenType};
+use crate::scanner::{Marker, ScanError, TScalarStyle, TokenType};
 use std::collections::BTreeMap;
 use std::f64;
 use std::i64;
@@ -368,7 +368,7 @@ impl Iterator for YamlIter {
 #[cfg(test)]
 mod test {
     use std::f64;
-    use yaml::*;
+    use crate::yaml::*;
     #[test]
     fn test_coerce() {
         let s = "---