From 69a54eebcf1c6eb7823b5d711e97fa4ca48bde24 Mon Sep 17 00:00:00 2001 From: Marc-Antoine Arnaud Date: Mon, 5 Nov 2018 17:17:27 +0100 Subject: [PATCH] format code --- .travis.yml | 7 ++-- yaserde/src/de/mod.rs | 2 +- yaserde/src/lib.rs | 4 +- yaserde/src/ser/mod.rs | 4 +- yaserde/tests/der_namespace.rs | 14 +++++-- yaserde/tests/der_option.rs | 8 ++-- yaserde/tests/der_type.rs | 9 +++-- yaserde/tests/deserializer.rs | 2 +- yaserde/tests/errors.rs | 2 +- yaserde/tests/se_namespace.rs | 44 +++++++++++++++------ yaserde/tests/se_option.rs | 2 +- yaserde/tests/se_type.rs | 11 +++--- yaserde/tests/serializer.rs | 2 +- yaserde_derive/src/attribute.rs | 4 +- yaserde_derive/src/de/expand_enum.rs | 4 +- yaserde_derive/src/de/expand_struct.rs | 4 +- yaserde_derive/src/field_type.rs | 23 ++++------- yaserde_derive/src/lib.rs | 2 +- yaserde_derive/src/ser/expand_enum.rs | 4 +- yaserde_derive/src/ser/expand_struct.rs | 52 ++++++++++++------------- 20 files changed, 113 insertions(+), 91 deletions(-) diff --git a/.travis.yml b/.travis.yml index b0bb005..64e718f 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,9 +1,10 @@ language: rust rust: - - 1.23.0 - - 1.24.0 - - 1.25.0 - 1.26.0 + - 1.27.0 + - 1.28.0 + - 1.29.0 + - 1.30.0 - stable - beta - nightly diff --git a/yaserde/src/de/mod.rs b/yaserde/src/de/mod.rs index bcb85d9..ef19789 100644 --- a/yaserde/src/de/mod.rs +++ b/yaserde/src/de/mod.rs @@ -1,6 +1,6 @@ use std::io::Read; -use xml::reader::{EventReader, ParserConfig, XmlEvent}; use xml::name::OwnedName; +use xml::reader::{EventReader, ParserConfig, XmlEvent}; use YaDeserialize; pub fn from_str(s: &str) -> Result { diff --git a/yaserde/src/lib.rs b/yaserde/src/lib.rs index 15631ae..7690ea3 100644 --- a/yaserde/src/lib.rs +++ b/yaserde/src/lib.rs @@ -74,7 +74,7 @@ pub trait Visitor<'de>: Sized { } macro_rules! serialize_type { - ($type: ty) => { + ($type:ty) => { impl YaSerialize for $type { fn serialize(&self, writer: &mut ser::Serializer) -> Result<(), String> { let content = format!("{}", self); @@ -112,7 +112,7 @@ fn default_visitor() { } macro_rules! test_type { - ($visitor: tt, $message: expr) => {{ + ($visitor:tt, $message:expr) => {{ let t = Test {}; assert_eq!(t.$visitor(""), Err($message.to_string())); }}; diff --git a/yaserde/src/ser/mod.rs b/yaserde/src/ser/mod.rs index 4af91f0..21a9e69 100644 --- a/yaserde/src/ser/mod.rs +++ b/yaserde/src/ser/mod.rs @@ -1,8 +1,8 @@ -use std::str; use std::io::{Cursor, Write}; -use xml::{EmitterConfig, EventWriter}; +use std::str; use xml; use xml::writer::XmlEvent; +use xml::{EmitterConfig, EventWriter}; use YaSerialize; pub fn to_string(model: &T) -> Result { diff --git a/yaserde/tests/der_namespace.rs b/yaserde/tests/der_namespace.rs index 98654f0..065e417 100644 --- a/yaserde/tests/der_namespace.rs +++ b/yaserde/tests/der_namespace.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Read; -use yaserde::YaDeserialize; use yaserde::de::from_str; +use yaserde::YaDeserialize; macro_rules! convert_and_validate { ($content: expr, $struct: tt, $model: expr) => { @@ -19,7 +19,11 @@ macro_rules! convert_and_validate { #[test] fn de_struct_namespace() { #[derive(YaDeserialize, PartialEq, Debug)] - #[yaserde(root = "book", prefix = "ns", namespace = "ns: http://www.sample.com/ns/domain")] + #[yaserde( + root = "book", + prefix = "ns", + namespace = "ns: http://www.sample.com/ns/domain" + )] pub struct Book { #[yaserde(prefix = "ns")] author: String, @@ -45,7 +49,11 @@ fn de_struct_namespace() { #[test] fn de_enum_namespace() { #[derive(YaDeserialize, PartialEq, Debug)] - #[yaserde(root = "root", prefix = "ns", namespace = "ns: http://www.sample.com/ns/domain")] + #[yaserde( + root = "root", + prefix = "ns", + namespace = "ns: http://www.sample.com/ns/domain" + )] pub enum XmlStruct { #[yaserde(prefix = "ns")] Item, diff --git a/yaserde/tests/der_option.rs b/yaserde/tests/der_option.rs index 67d0222..7346dba 100644 --- a/yaserde/tests/der_option.rs +++ b/yaserde/tests/der_option.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Read; -use yaserde::YaDeserialize; use yaserde::de::from_str; +use yaserde::YaDeserialize; macro_rules! convert_and_validate { ($type: ty, $value: expr, $content: expr) => {{ @@ -20,7 +20,8 @@ macro_rules! convert_and_validate { let model = Data { item: $value }; let source = if let Some(content) = $content { - String::from("") + content + String::from("") + + content + "" } else { String::from("") @@ -43,7 +44,8 @@ macro_rules! convert_and_validate_for_attribute { let model = Data { item: $value }; let source = if let Some(content) = $content { - String::from("" } else { String::from("") diff --git a/yaserde/tests/der_type.rs b/yaserde/tests/der_type.rs index ba96c0d..e6f82d1 100644 --- a/yaserde/tests/der_type.rs +++ b/yaserde/tests/der_type.rs @@ -6,11 +6,11 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Read; -use yaserde::YaDeserialize; use yaserde::de::from_str; +use yaserde::YaDeserialize; macro_rules! convert_and_validate { - ($type: ty, $value: expr, $content: expr) => {{ + ($type:ty, $value:expr, $content:expr) => {{ #[derive(YaDeserialize, PartialEq, Debug)] #[yaserde(root = "data")] pub struct Data { @@ -19,7 +19,8 @@ macro_rules! convert_and_validate { let model = Data { item: $value }; - let content = String::from("") + $content + let content = String::from("") + + $content + ""; let loaded: Result = from_str(&content); @@ -28,7 +29,7 @@ macro_rules! convert_and_validate { } macro_rules! convert_and_validate_for_attribute { - ($type: ty, $value: expr, $content: expr) => {{ + ($type:ty, $value:expr, $content:expr) => {{ #[derive(YaDeserialize, PartialEq, Debug)] #[yaserde(root = "data")] pub struct Data { diff --git a/yaserde/tests/deserializer.rs b/yaserde/tests/deserializer.rs index 20a0ac2..2b94548 100644 --- a/yaserde/tests/deserializer.rs +++ b/yaserde/tests/deserializer.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Read; -use yaserde::YaDeserialize; use yaserde::de::from_str; +use yaserde::YaDeserialize; macro_rules! convert_and_validate { ($content: expr, $struct: tt, $model: expr) => { diff --git a/yaserde/tests/errors.rs b/yaserde/tests/errors.rs index d911f2c..ac68639 100644 --- a/yaserde/tests/errors.rs +++ b/yaserde/tests/errors.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Read; -use yaserde::YaDeserialize; use yaserde::de::from_str; +use yaserde::YaDeserialize; #[test] fn de_no_content() { diff --git a/yaserde/tests/se_namespace.rs b/yaserde/tests/se_namespace.rs index 7d4e39a..0c65936 100644 --- a/yaserde/tests/se_namespace.rs +++ b/yaserde/tests/se_namespace.rs @@ -6,11 +6,11 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Write; -use yaserde::YaSerialize; use yaserde::ser::to_string; +use yaserde::YaSerialize; macro_rules! convert_and_validate { - ($model: expr, $content: expr) => { + ($model:expr, $content:expr) => { let data: Result = to_string(&$model); assert_eq!(data, Ok(String::from($content))); }; @@ -19,7 +19,11 @@ macro_rules! convert_and_validate { #[test] fn ser_struct_namespace() { #[derive(YaSerialize, PartialEq, Debug)] - #[yaserde(root = "root", prefix = "ns", namespace = "ns: http://www.sample.com/ns/domain")] + #[yaserde( + root = "root", + prefix = "ns", + namespace = "ns: http://www.sample.com/ns/domain" + )] pub struct XmlStruct { #[yaserde(prefix = "ns")] item: String, @@ -36,7 +40,11 @@ fn ser_struct_namespace() { #[test] fn ser_enum_namespace() { #[derive(YaSerialize, PartialEq, Debug)] - #[yaserde(root = "root", prefix = "ns", namespace = "ns: http://www.sample.com/ns/domain")] + #[yaserde( + root = "root", + prefix = "ns", + namespace = "ns: http://www.sample.com/ns/domain" + )] pub enum XmlStruct { #[yaserde(prefix = "ns")] Item, @@ -51,8 +59,11 @@ fn ser_enum_namespace() { #[test] fn ser_struct_multi_namespace() { #[derive(YaSerialize, PartialEq, Debug)] - #[yaserde(root = "root", namespace = "ns1: http://www.sample.com/ns/domain1", - namespace = "ns2: http://www.sample.com/ns/domain2")] + #[yaserde( + root = "root", + namespace = "ns1: http://www.sample.com/ns/domain1", + namespace = "ns2: http://www.sample.com/ns/domain2" + )] pub struct XmlStruct { #[yaserde(prefix = "ns1")] item_1: String, @@ -72,8 +83,11 @@ fn ser_struct_multi_namespace() { #[test] fn ser_enum_multi_namespace() { #[derive(YaSerialize, PartialEq, Debug)] - #[yaserde(root = "root", namespace = "ns1: http://www.sample.com/ns/domain1", - namespace = "ns2: http://www.sample.com/ns/domain2")] + #[yaserde( + root = "root", + namespace = "ns1: http://www.sample.com/ns/domain1", + namespace = "ns2: http://www.sample.com/ns/domain2" + )] pub enum XmlStruct { #[yaserde(prefix = "ns1")] Item1, @@ -92,8 +106,11 @@ fn ser_enum_multi_namespace() { #[test] fn ser_struct_attribute_namespace() { #[derive(YaSerialize, PartialEq, Debug)] - #[yaserde(root = "root", namespace = "ns1: http://www.sample.com/ns/domain1", - namespace = "ns2: http://www.sample.com/ns/domain2")] + #[yaserde( + root = "root", + namespace = "ns1: http://www.sample.com/ns/domain1", + namespace = "ns2: http://www.sample.com/ns/domain2" + )] pub struct XmlStruct { #[yaserde(prefix = "ns1")] item_1: String, @@ -113,8 +130,11 @@ fn ser_struct_attribute_namespace() { #[test] fn ser_struct_default_namespace() { #[derive(YaSerialize, PartialEq, Debug)] - #[yaserde(root = "tt", namespace = "http://www.w3.org/ns/ttml", - namespace = "ttm: http://www.w3.org/ns/ttml#metadata")] + #[yaserde( + root = "tt", + namespace = "http://www.w3.org/ns/ttml", + namespace = "ttm: http://www.w3.org/ns/ttml#metadata" + )] pub struct XmlStruct { item: String, } diff --git a/yaserde/tests/se_option.rs b/yaserde/tests/se_option.rs index 6f2f638..974529b 100644 --- a/yaserde/tests/se_option.rs +++ b/yaserde/tests/se_option.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Write; -use yaserde::YaSerialize; use yaserde::ser::to_string; +use yaserde::YaSerialize; macro_rules! convert_and_validate { ($type: ty, $value: expr, $content: expr) => {{ diff --git a/yaserde/tests/se_type.rs b/yaserde/tests/se_type.rs index d44cf7a..cda4430 100644 --- a/yaserde/tests/se_type.rs +++ b/yaserde/tests/se_type.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Write; -use yaserde::YaSerialize; use yaserde::ser::to_string; +use yaserde::YaSerialize; macro_rules! convert_and_validate { ($type: ty, $value: expr, $content: expr) => {{ @@ -20,11 +20,12 @@ macro_rules! convert_and_validate { let data: Result = to_string(&model); let content = if $content == "" { - String::from("") - } else { - String::from("") + $content + String::from("") + } else { + String::from("") + + $content + "" - }; + }; assert_eq!(data, Ok(content)); }}; } diff --git a/yaserde/tests/serializer.rs b/yaserde/tests/serializer.rs index c52e59f..68337b8 100644 --- a/yaserde/tests/serializer.rs +++ b/yaserde/tests/serializer.rs @@ -6,8 +6,8 @@ extern crate yaserde; extern crate yaserde_derive; use std::io::Write; -use yaserde::YaSerialize; use yaserde::ser::to_string; +use yaserde::YaSerialize; macro_rules! convert_and_validate { ($model: expr, $content: expr) => { diff --git a/yaserde_derive/src/attribute.rs b/yaserde_derive/src/attribute.rs index 30f88c0..16ec95b 100644 --- a/yaserde_derive/src/attribute.rs +++ b/yaserde_derive/src/attribute.rs @@ -1,6 +1,6 @@ -use proc_macro2::TokenTree; -use proc_macro2::Delimiter; use proc_macro2::token_stream::IntoIter; +use proc_macro2::Delimiter; +use proc_macro2::TokenTree; use std::collections::BTreeMap; use syn::Attribute; diff --git a/yaserde_derive/src/de/expand_enum.rs b/yaserde_derive/src/de/expand_enum.rs index 585f153..f967a8b 100644 --- a/yaserde_derive/src/de/expand_enum.rs +++ b/yaserde_derive/src/de/expand_enum.rs @@ -1,11 +1,11 @@ use attribute::*; use field_type::*; +use proc_macro2::{Span, TokenStream}; use quote::TokenStreamExt; use std::collections::BTreeMap; +use syn::DataEnum; use syn::Fields; use syn::Ident; -use syn::DataEnum; -use proc_macro2::{Span, TokenStream}; pub fn parse( data_enum: &DataEnum, diff --git a/yaserde_derive/src/de/expand_struct.rs b/yaserde_derive/src/de/expand_struct.rs index bdc3acc..bdb8461 100644 --- a/yaserde_derive/src/de/expand_struct.rs +++ b/yaserde_derive/src/de/expand_struct.rs @@ -1,10 +1,10 @@ use attribute::*; use field_type::*; +use proc_macro2::{Span, TokenStream}; use quote::TokenStreamExt; use std::collections::BTreeMap; -use syn::Ident; use syn::DataStruct; -use proc_macro2::{Span, TokenStream}; +use syn::Ident; pub fn parse( data_struct: &DataStruct, diff --git a/yaserde_derive/src/field_type.rs b/yaserde_derive/src/field_type.rs index 0fec2a9..ad8f9a3 100644 --- a/yaserde_derive/src/field_type.rs +++ b/yaserde_derive/src/field_type.rs @@ -36,20 +36,15 @@ impl FieldType { "u64" => Some(FieldType::FieldTypeU64), "f32" => Some(FieldType::FieldTypeF32), "f64" => Some(FieldType::FieldTypeF64), - "Option" => get_sub_type(t).map(|data_type| { - FieldType::FieldTypeOption { - data_type: Box::new(FieldType::from_ident(&data_type).unwrap()), - } + "Option" => get_sub_type(t).map(|data_type| FieldType::FieldTypeOption { + data_type: Box::new(FieldType::from_ident(&data_type).unwrap()), }), - "Vec" => get_sub_type(t).map(|data_type| { - FieldType::FieldTypeVec { - data_type: Box::new(FieldType::from_ident(&data_type).unwrap()), - } + "Vec" => get_sub_type(t).map(|data_type| FieldType::FieldTypeVec { + data_type: Box::new(FieldType::from_ident(&data_type).unwrap()), }), - _struct_name => { - Some(FieldType::FieldTypeStruct { + _struct_name => Some(FieldType::FieldTypeStruct { struct_name: t.ident.clone(), - })}, + }), } } } @@ -62,11 +57,9 @@ pub fn get_field_type(field: &syn::Field) -> Option { } match path.path.segments.first() { Some(Pair::End(t)) => FieldType::from_ident(t), - _ => { - None - }, + _ => None, } - }, + } _ => None, } } diff --git a/yaserde_derive/src/lib.rs b/yaserde_derive/src/lib.rs index 3d459c8..527a439 100644 --- a/yaserde_derive/src/lib.rs +++ b/yaserde_derive/src/lib.rs @@ -7,8 +7,8 @@ extern crate quote; extern crate syn; mod attribute; -mod field_type; mod de; +mod field_type; mod ser; use proc_macro::TokenStream; diff --git a/yaserde_derive/src/ser/expand_enum.rs b/yaserde_derive/src/ser/expand_enum.rs index 2b4c60f..08ca73e 100644 --- a/yaserde_derive/src/ser/expand_enum.rs +++ b/yaserde_derive/src/ser/expand_enum.rs @@ -1,11 +1,11 @@ use attribute::*; use field_type::*; +use proc_macro2::{Span, TokenStream}; use quote::TokenStreamExt; use std::collections::BTreeMap; +use syn::DataEnum; use syn::Fields; use syn::Ident; -use syn::DataEnum; -use proc_macro2::{Span, TokenStream}; pub fn serialize( data_enum: &DataEnum, diff --git a/yaserde_derive/src/ser/expand_struct.rs b/yaserde_derive/src/ser/expand_struct.rs index fe6e757..76fd7d0 100644 --- a/yaserde_derive/src/ser/expand_struct.rs +++ b/yaserde_derive/src/ser/expand_struct.rs @@ -1,11 +1,11 @@ use attribute::*; use field_type::*; +use proc_macro2::{Span, TokenStream}; use quote::TokenStreamExt; use std::collections::BTreeMap; -use syn::Ident; -use syn::DataStruct; -use proc_macro2::{Span, TokenStream}; use std::string::ToString; +use syn::DataStruct; +use syn::Ident; pub fn serialize( data_struct: &DataStruct, @@ -93,20 +93,18 @@ pub fn serialize( struct_start_event }; }), - Some(&FieldType::FieldTypeVec{..}) => { - Some(quote!{ - for item in &self.#label { - let start_event = XmlEvent::start_element(#label_name); - let _ret = writer.write(start_event); + Some(&FieldType::FieldTypeVec { .. }) => Some(quote!{ + for item in &self.#label { + let start_event = XmlEvent::start_element(#label_name); + let _ret = writer.write(start_event); - let data_event = XmlEvent::characters(item); - let _ret = writer.write(data_event); + let data_event = XmlEvent::characters(item); + let _ret = writer.write(data_event); - let end_event = XmlEvent::end_element(); - let _ret = writer.write(end_event); - } - }) - } + let end_event = XmlEvent::end_element(); + let _ret = writer.write(end_event); + } + }), _ => None, } } @@ -247,7 +245,7 @@ pub fn serialize( let _ret = writer.write(end_event); } }), - Some(&FieldType::FieldTypeVec{ .. }) => Some(quote!{ + Some(&FieldType::FieldTypeVec { .. }) => Some(quote!{ if let Some(ref items) = &self.#label { for item in items.iter() { let start_event = XmlEvent::start_element(#label_name); @@ -265,17 +263,15 @@ pub fn serialize( _ => None, } } - Some(FieldType::FieldTypeStruct { .. }) => { - Some(quote!{ - writer.set_skip_start_end(false); - match self.#label.serialize(writer) { - Ok(()) => {}, - Err(msg) => { - return Err(msg); - }, - }; - }) - }, + Some(FieldType::FieldTypeStruct { .. }) => Some(quote!{ + writer.set_skip_start_end(false); + match self.#label.serialize(writer) { + Ok(()) => {}, + Err(msg) => { + return Err(msg); + }, + }; + }), Some(FieldType::FieldTypeVec { data_type }) => { let dt = Box::into_raw(data_type); match unsafe { dt.as_ref() } { @@ -344,7 +340,7 @@ pub fn serialize( unimplemented!(); } } - }, + } None => None, } })