mikros/serde
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Remove the no_std implementation of collect_str

Browse Source
This commit is contained in:
David Tolnay 2017-04-04 19:04:47 -07:00
parent 92bc23e484
commit f05368ed24
No known key found for this signature in database
GPG Key ID: F9BA143B95FF6D82
2 changed files with 11 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
serde/src/ser/mod.rs
View File

@ -654,8 +654,9 @@ pub trait Serializer: Sized {
/// Serialize a string produced by an implementation of `Display`.
///
/// The default implementation returns an error unconditionally when
/// compiled with `no_std`.
/// Serializers that use `no_std` are required to provide an implementation
/// of this method. If no more sensible behavior is possible, the
/// implementation is expected to return an error.
///
/// ```rust
/// # use serde::{Serialize, Serializer};
@ -676,13 +677,7 @@ pub trait Serializer: Sized {
/// ```
#[cfg(not(any(feature = "std", feature = "collections")))]
fn collect_str<T: ?Sized>(self, value: &T) -> Result<Self::Ok, Self::Error>
where T: Display
{
// TODO https://github.com/serde-rs/serde/issues/805
// Remove this impl and force no_std formats to implement collect_str.
let _ = value;
Err(Error::custom("this no_std format does not support serializing strings with collect_str"))
}
where T: Display;
}
/// Returned from `Serializer::serialize_seq` and

7
serde/src/ser/private.rs
View File

@ -312,4 +312,11 @@ impl<S> Serializer for TaggedSerializer<S>
try!(map.serialize_key(inner_variant));
Ok(SerializeStructVariantAsMapValue::new(map, inner_variant, len))
}
#[cfg(not(any(feature = "std", feature = "collections")))]
fn collect_str<T: ?Sized>(self, _: &T) -> Result<Self::Ok, Self::Error>
where T: Display
{
Err(self.bad_type(Unsupported::String))
}
}