diff options
| -rw-r--r-- | src/builder.rs | 2 | ||||
| -rw-r--r-- | src/date.rs | 8 | ||||
| -rw-r--r-- | src/de.rs | 4 | ||||
| -rw-r--r-- | src/events/binary_reader.rs | 35 | ||||
| -rw-r--r-- | src/events/xml_reader.rs | 8 | ||||
| -rw-r--r-- | src/events/xml_writer.rs | 3 | ||||
| -rw-r--r-- | src/ser.rs | 16 | ||||
| -rw-r--r-- | src/value.rs | 28 |
8 files changed, 51 insertions, 53 deletions
diff --git a/src/builder.rs b/src/builder.rs index d4d9364..ca9deb5 100644 --- a/src/builder.rs +++ b/src/builder.rs @@ -11,7 +11,7 @@ pub struct Builder<T> { impl<T: Iterator<Item = Result<Event, Error>>> Builder<T> { pub fn new(stream: T) -> Builder<T> { Builder { - stream: stream, + stream, token: None, } } diff --git a/src/date.rs b/src/date.rs index 4ee1dae..f83b1af 100644 --- a/src/date.rs +++ b/src/date.rs @@ -25,7 +25,7 @@ impl Date { pub(crate) fn from_seconds_since_plist_epoch(timestamp: f64) -> Result<Date, Error> { // `timestamp` is the number of seconds since the plist epoch of 1/1/2001 00:00:00. // `PLIST_EPOCH_UNIX_TIMESTAMP` is the unix timestamp of the plist epoch. - const PLIST_EPOCH_UNIX_TIMESTAMP: u64 = 978307200; + const PLIST_EPOCH_UNIX_TIMESTAMP: u64 = 978_307_200; let plist_epoch = UNIX_EPOCH + Duration::from_secs(PLIST_EPOCH_UNIX_TIMESTAMP); if !timestamp.is_finite() { @@ -58,13 +58,13 @@ impl fmt::Debug for Date { impl From<SystemTime> for Date { fn from(date: SystemTime) -> Self { - Date { inner: date.into() } + Date { inner: date } } } impl Into<SystemTime> for Date { fn into(self) -> SystemTime { - self.inner.into() + self.inner } } @@ -76,7 +76,7 @@ pub mod serde_impls { use Date; - pub const DATE_NEWTYPE_STRUCT_NAME: &'static str = "PLIST-DATE"; + pub const DATE_NEWTYPE_STRUCT_NAME: &str = "PLIST-DATE"; impl Serialize for Date { fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error> @@ -322,8 +322,8 @@ where len: Option<usize>, ) -> MapAndSeqAccess<'a, I> { MapAndSeqAccess { - de: de, - is_struct: is_struct, + de, + is_struct, remaining: len, } } diff --git a/src/events/binary_reader.rs b/src/events/binary_reader.rs index 553040d..6d31518 100644 --- a/src/events/binary_reader.rs +++ b/src/events/binary_reader.rs @@ -54,7 +54,7 @@ impl<R: Read + Seek> BinaryReader<R> { BinaryReader { stack: Vec::new(), object_offsets: Vec::new(), - reader: reader, + reader, ref_size: 0, finished: false, max_allocation_bytes: 0, @@ -127,10 +127,10 @@ impl<R: Read + Seek> BinaryReader<R> { let mut ints = self.allocate_vec(len, size as usize)?; for _ in 0..len { match size { - 1 => ints.push(self.reader.read_u8()? as u64), - 2 => ints.push(self.reader.read_u16::<BigEndian>()? as u64), - 4 => ints.push(self.reader.read_u32::<BigEndian>()? as u64), - 8 => ints.push(self.reader.read_u64::<BigEndian>()? as u64), + 1 => ints.push(self.reader.read_u8()?.into()), + 2 => ints.push(self.reader.read_u16::<BigEndian>()?.into()), + 4 => ints.push(self.reader.read_u32::<BigEndian>()?.into()), + 8 => ints.push(self.reader.read_u64::<BigEndian>()?), _ => return Err(Error::InvalidData), } } @@ -146,14 +146,14 @@ impl<R: Read + Seek> BinaryReader<R> { if (len & 0x0f) == 0x0f { let len_power_of_two = self.reader.read_u8()? & 0x03; Ok(match len_power_of_two { - 0 => self.reader.read_u8()? as u64, - 1 => self.reader.read_u16::<BigEndian>()? as u64, - 2 => self.reader.read_u32::<BigEndian>()? as u64, + 0 => self.reader.read_u8()?.into(), + 1 => self.reader.read_u16::<BigEndian>()?.into(), + 2 => self.reader.read_u32::<BigEndian>()?.into(), 3 => self.reader.read_u64::<BigEndian>()?, _ => return Err(Error::InvalidData), }) } else { - Ok(len as u64) + Ok(len.into()) } } @@ -166,8 +166,7 @@ impl<R: Read + Seek> BinaryReader<R> { fn seek_to_object(&mut self, object_ref: u64) -> Result<u64, Error> { let object_ref = u64_to_usize(object_ref)?; - let offset = *self - .object_offsets + let offset = *self.object_offsets .get(object_ref) .ok_or(Error::InvalidData)?; Ok(self.reader.seek(SeekFrom::Start(offset))?) @@ -214,17 +213,19 @@ impl<R: Read + Seek> BinaryReader<R> { (0x0, 0x08) => Some(Event::BooleanValue(false)), (0x0, 0x09) => Some(Event::BooleanValue(true)), (0x0, 0x0f) => return Err(Error::InvalidData), // fill - (0x1, 0) => Some(Event::IntegerValue(self.reader.read_u8()? as i64)), + (0x1, 0) => Some(Event::IntegerValue(self.reader.read_u8()?.into())), (0x1, 1) => Some(Event::IntegerValue( - self.reader.read_u16::<BigEndian>()? as i64 + self.reader.read_u16::<BigEndian>()?.into(), )), (0x1, 2) => Some(Event::IntegerValue( - self.reader.read_u32::<BigEndian>()? as i64 + self.reader.read_u32::<BigEndian>()?.into(), )), (0x1, 3) => Some(Event::IntegerValue(self.reader.read_i64::<BigEndian>()?)), (0x1, 4) => return Err(Error::InvalidData), // 128 bit int (0x1, _) => return Err(Error::InvalidData), // variable length int - (0x2, 2) => Some(Event::RealValue(self.reader.read_f32::<BigEndian>()? as f64)), + (0x2, 2) => Some(Event::RealValue( + self.reader.read_f32::<BigEndian>()?.into(), + )), (0x2, 3) => Some(Event::RealValue(self.reader.read_f64::<BigEndian>()?)), (0x2, _) => return Err(Error::InvalidData), // odd length float (0x3, 3) => { @@ -267,7 +268,7 @@ impl<R: Read + Seek> BinaryReader<R> { self.stack.push(StackItem { ty: StackType::Array, - object_refs: object_refs, + object_refs, }); Some(Event::StartArray(Some(len))) @@ -288,7 +289,7 @@ impl<R: Read + Seek> BinaryReader<R> { self.stack.push(StackItem { ty: StackType::Dict, - object_refs: object_refs, + object_refs, }); Some(Event::StartDictionary(Some(len as u64))) diff --git a/src/events/xml_reader.rs b/src/events/xml_reader.rs index 6bb9936..e2fff6a 100644 --- a/src/events/xml_reader.rs +++ b/src/events/xml_reader.rs @@ -74,11 +74,9 @@ impl<R: Read> XmlReader<R> { })) } "date" => { - return Some( - self.read_content(|s| { - Ok(Event::DateValue(Date::from_rfc3339(&s)?)) - }), - ) + return Some(self.read_content(|s| { + Ok(Event::DateValue(Date::from_rfc3339(&s)?)) + })) } "integer" => { return Some(self.read_content(|s| match FromStr::from_str(&s) { diff --git a/src/events/xml_writer.rs b/src/events/xml_writer.rs index a2df5bf..dde1e6e 100644 --- a/src/events/xml_writer.rs +++ b/src/events/xml_writer.rs @@ -120,8 +120,7 @@ impl<W: Write> Writer for XmlWriter<W> { }; return Ok(()); } - Some(Element::Dictionary(DictionaryState::ExpectValue)) => self - .stack + Some(Element::Dictionary(DictionaryState::ExpectValue)) => self.stack .push(Element::Dictionary(DictionaryState::ExpectKey)), Some(other) => self.stack.push(other), None => { @@ -18,7 +18,7 @@ pub struct Serializer<W: Writer> { impl<W: Writer> Serializer<W> { pub fn new(writer: W) -> Serializer<W> { - Serializer { writer: writer } + Serializer { writer } } fn emit(&mut self, event: Event) -> Result<(), Error> { @@ -60,15 +60,15 @@ impl<'a, W: Writer> ser::Serializer for &'a mut Serializer<W> { } fn serialize_i8(self, v: i8) -> Result<(), Self::Error> { - self.serialize_i64(v as i64) + self.serialize_i64(v.into()) } fn serialize_i16(self, v: i16) -> Result<(), Self::Error> { - self.serialize_i64(v as i64) + self.serialize_i64(v.into()) } fn serialize_i32(self, v: i32) -> Result<(), Self::Error> { - self.serialize_i64(v as i64) + self.serialize_i64(v.into()) } fn serialize_i64(self, v: i64) -> Result<(), Self::Error> { @@ -76,15 +76,15 @@ impl<'a, W: Writer> ser::Serializer for &'a mut Serializer<W> { } fn serialize_u8(self, v: u8) -> Result<(), Self::Error> { - self.serialize_u64(v as u64) + self.serialize_u64(v.into()) } fn serialize_u16(self, v: u16) -> Result<(), Self::Error> { - self.serialize_u64(v as u64) + self.serialize_u64(v.into()) } fn serialize_u32(self, v: u32) -> Result<(), Self::Error> { - self.serialize_u64(v as u64) + self.serialize_u64(v.into()) } fn serialize_u64(self, v: u64) -> Result<(), Self::Error> { @@ -92,7 +92,7 @@ impl<'a, W: Writer> ser::Serializer for &'a mut Serializer<W> { } fn serialize_f32(self, v: f32) -> Result<(), Self::Error> { - self.serialize_f64(v as f64) + self.serialize_f64(v.into()) } fn serialize_f64(self, v: f64) -> Result<(), Self::Error> { diff --git a/src/value.rs b/src/value.rs index bb422e7..e0a95e5 100644 --- a/src/value.rs +++ b/src/value.rs @@ -222,7 +222,7 @@ impl From<f64> for Value { impl From<f32> for Value { fn from(from: f32) -> Value { - Value::Real(from as f64) + Value::Real(from.into()) } } @@ -234,37 +234,37 @@ impl From<i64> for Value { impl From<i32> for Value { fn from(from: i32) -> Value { - Value::Integer(from as i64) + Value::Integer(from.into()) } } impl From<i16> for Value { fn from(from: i16) -> Value { - Value::Integer(from as i64) + Value::Integer(from.into()) } } impl From<i8> for Value { fn from(from: i8) -> Value { - Value::Integer(from as i64) + Value::Integer(from.into()) } } impl From<u32> for Value { fn from(from: u32) -> Value { - Value::Integer(from as i64) + Value::Integer(from.into()) } } impl From<u16> for Value { fn from(from: u16) -> Value { - Value::Integer(from as i64) + Value::Integer(from.into()) } } impl From<u8> for Value { fn from(from: u8) -> Value { - Value::Integer(from as i64) + Value::Integer(from.into()) } } @@ -276,7 +276,7 @@ impl<'a> From<&'a f64> for Value { impl<'a> From<&'a f32> for Value { fn from(from: &'a f32) -> Value { - Value::Real(*from as f64) + Value::Real((*from).into()) } } @@ -288,37 +288,37 @@ impl<'a> From<&'a i64> for Value { impl<'a> From<&'a i32> for Value { fn from(from: &'a i32) -> Value { - Value::Integer(*from as i64) + Value::Integer((*from).into()) } } impl<'a> From<&'a i16> for Value { fn from(from: &'a i16) -> Value { - Value::Integer(*from as i64) + Value::Integer((*from).into()) } } impl<'a> From<&'a i8> for Value { fn from(from: &'a i8) -> Value { - Value::Integer(*from as i64) + Value::Integer((*from).into()) } } impl<'a> From<&'a u32> for Value { fn from(from: &'a u32) -> Value { - Value::Integer(*from as i64) + Value::Integer((*from).into()) } } impl<'a> From<&'a u16> for Value { fn from(from: &'a u16) -> Value { - Value::Integer(*from as i64) + Value::Integer((*from).into()) } } impl<'a> From<&'a u8> for Value { fn from(from: &'a u8) -> Value { - Value::Integer(*from as i64) + Value::Integer((*from).into()) } } |
