From dc407a6833b8358855fe5fb949b4253874311f47 Mon Sep 17 00:00:00 2001 From: Alan Du Date: Thu, 1 Jun 2017 20:35:39 +0100 Subject: Truncate fractional seconds to picoseconds Close https://github.com/alexcrichton/toml-rs/issues/186 --- src/datetime.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/datetime.rs b/src/datetime.rs index f39d9fa..810a4d7 100644 --- a/src/datetime.rs +++ b/src/datetime.rs @@ -218,7 +218,10 @@ impl FromStr for Datetime { return Err(DatetimeParseError { _private: () }) } chars = whole[end..].chars(); - match format!("0.{}", &whole[..end]).parse() { + + // truncate to picoseconds precision + let last = if end > 12 { 12 } else { end }; + match format!("0.{}", &whole[..last]).parse() { Ok(f) => f, Err(_) => return Err(DatetimeParseError { _private: () }), } -- cgit v1.2.3 From 69576569a7283ba874893fc71b6bff9407d823a6 Mon Sep 17 00:00:00 2001 From: Alan Du Date: Thu, 1 Jun 2017 20:54:17 +0100 Subject: Allow serializing keys with \n in them Use special quoted form Closes https://github.com/alexcrichton/toml-rs/issues/185 --- src/ser.rs | 8 -------- 1 file changed, 8 deletions(-) (limited to 'src') diff --git a/src/ser.rs b/src/ser.rs index 8f6b366..9fa0678 100644 --- a/src/ser.rs +++ b/src/ser.rs @@ -107,9 +107,6 @@ pub enum Error { /// attempted where the key of a map was not a string. KeyNotString, - /// Keys in maps are not allowed to have newlines. - KeyNewline, - /// Arrays in TOML must have a homogenous type, but a heterogeneous array /// was emitted. ArrayMixedType, @@ -624,9 +621,6 @@ impl<'a, 'b> ser::SerializeMap for SerializeTable<'a, 'b> { SerializeTable::Table { ref mut key, .. } => { key.truncate(0); *key = input.serialize(StringExtractor)?; - if key.contains('\n') { - return Err(Error::KeyNewline) - } } } Ok(()) @@ -1047,7 +1041,6 @@ impl fmt::Display for Error { match *self { Error::UnsupportedType => "unsupported Rust type".fmt(f), Error::KeyNotString => "map key was not a string".fmt(f), - Error::KeyNewline => "map keys cannot contain newlines".fmt(f), Error::ArrayMixedType => "arrays cannot have mixed types".fmt(f), Error::ValueAfterTable => "values must be emitted before tables".fmt(f), Error::DateInvalid => "a serialized date was invalid".fmt(f), @@ -1064,7 +1057,6 @@ impl error::Error for Error { match *self { Error::UnsupportedType => "unsupported Rust type", Error::KeyNotString => "map key was not a string", - Error::KeyNewline => "map keys cannot contain newlines", Error::ArrayMixedType => "arrays cannot have mixed types", Error::ValueAfterTable => "values must be emitted before tables", Error::DateInvalid => "a serialized date was invalid", -- cgit v1.2.3 From e4fdd0e4e49842d50f75e4d80f555fa783c0709f Mon Sep 17 00:00:00 2001 From: Alan Du Date: Thu, 1 Jun 2017 22:47:24 +0100 Subject: Add toml::ser::Error::KeyNewline back in Avoid a breaking change --- src/ser.rs | 6 ++++++ 1 file changed, 6 insertions(+) (limited to 'src') diff --git a/src/ser.rs b/src/ser.rs index 9fa0678..77db867 100644 --- a/src/ser.rs +++ b/src/ser.rs @@ -107,6 +107,10 @@ pub enum Error { /// attempted where the key of a map was not a string. KeyNotString, + /// An error that we never omit but keep for backwards compatibility + #[doc(hidden)] + KeyNewline, + /// Arrays in TOML must have a homogenous type, but a heterogeneous array /// was emitted. ArrayMixedType, @@ -1047,6 +1051,7 @@ impl fmt::Display for Error { Error::NumberInvalid => "a serialized number was invalid".fmt(f), Error::UnsupportedNone => "unsupported None value".fmt(f), Error::Custom(ref s) => s.fmt(f), + Error::KeyNewline => unreachable!(), Error::__Nonexhaustive => panic!(), } } @@ -1063,6 +1068,7 @@ impl error::Error for Error { Error::NumberInvalid => "a serialized number was invalid", Error::UnsupportedNone => "unsupported None value", Error::Custom(_) => "custom error", + Error::KeyNewline => unreachable!(), Error::__Nonexhaustive => panic!(), } } -- cgit v1.2.3 From 2dfc9fedd30bf3d40c4c3e97adc06fe3aad34361 Mon Sep 17 00:00:00 2001 From: Alan Du Date: Thu, 1 Jun 2017 23:22:35 +0100 Subject: Store fractional seconds as a u32 instead of a f64 Drops precision down to the nanoseconds --- src/datetime.rs | 47 +++++++++++++++++++++++------------------------ 1 file changed, 23 insertions(+), 24 deletions(-) (limited to 'src') diff --git a/src/datetime.rs b/src/datetime.rs index 810a4d7..83b5c0b 100644 --- a/src/datetime.rs +++ b/src/datetime.rs @@ -55,7 +55,7 @@ struct Time { hour: u8, minute: u8, second: u8, - secfract: f64, + nanosecond: u32, } #[derive(PartialEq, Clone)] @@ -97,9 +97,9 @@ impl fmt::Display for Date { impl fmt::Display for Time { fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { write!(f, "{:02}:{:02}:{:02}", self.hour, self.minute, self.second)?; - if self.secfract != 0.0 { - let s = format!("{}", self.secfract); - write!(f, "{}", s.trim_left_matches("0"))?; + if self.nanosecond != 0 { + let s = format!("{:09}", self.nanosecond); + write!(f, ".{}", s.trim_right_matches('0'))?; } Ok(()) } @@ -199,41 +199,37 @@ impl FromStr for Datetime { let s1 = digit(&mut chars)?; let s2 = digit(&mut chars)?; - let secfract = if chars.clone().next() == Some('.') { + let mut nanosecond = 0; + if chars.clone().next() == Some('.') { chars.next(); - let mut first = true; let whole = chars.as_str(); + let mut end = whole.len(); - for (i, c) in whole.char_indices() { - match c { - '0' ... '9' => {} + for (i, byte) in whole.bytes().enumerate() { + match byte { + b'0' ... b'9' => { + if i < 9 { + let p = 10_u32.pow(8 - i as u32); + nanosecond += p * (byte - b'0') as u32; + } + } _ => { end = i; - break + break; } } - first = false; } - if first { + if end == 0 { return Err(DatetimeParseError { _private: () }) } chars = whole[end..].chars(); - - // truncate to picoseconds precision - let last = if end > 12 { 12 } else { end }; - match format!("0.{}", &whole[..last]).parse() { - Ok(f) => f, - Err(_) => return Err(DatetimeParseError { _private: () }), - } - } else { - 0.0 - }; + } let time = Time { hour: h1 * 10 + h2, minute: m1 * 10 + m2, second: s1 * 10 + s2, - secfract: secfract, + nanosecond: nanosecond, }; if time.hour > 24 { @@ -242,7 +238,10 @@ impl FromStr for Datetime { if time.minute > 59 { return Err(DatetimeParseError { _private: () }) } - if time.second > 60 { + if time.second > 59 { + return Err(DatetimeParseError { _private: () }) + } + if time.nanosecond > 999_999_999 { return Err(DatetimeParseError { _private: () }) } -- cgit v1.2.3