Merge pull request #477 from serde-rs/iter

Fix clippy lint about looping over iter()
This commit is contained in:
Oliver Schneider
2016-08-04 08:48:35 +02:00
committed by GitHub
2 changed files with 3 additions and 3 deletions
+2 -2
View File
@@ -157,7 +157,7 @@ impl<T> Serialize for [T]
where S: Serializer, where S: Serializer,
{ {
let mut state = try!(serializer.serialize_seq(Some(self.len()))); let mut state = try!(serializer.serialize_seq(Some(self.len())));
for e in self.iter() { for e in self {
try!(serializer.serialize_seq_elt(&mut state, e)); try!(serializer.serialize_seq_elt(&mut state, e));
} }
serializer.serialize_seq_end(state) serializer.serialize_seq_end(state)
@@ -174,7 +174,7 @@ macro_rules! array_impls {
where S: Serializer, where S: Serializer,
{ {
let mut state = try!(serializer.serialize_seq_fixed_size($len)); let mut state = try!(serializer.serialize_seq_fixed_size($len));
for e in self.iter() { for e in self {
try!(serializer.serialize_seq_elt(&mut state, e)); try!(serializer.serialize_seq_elt(&mut state, e));
} }
serializer.serialize_seq_end(state) serializer.serialize_seq_end(state)
+1 -1
View File
@@ -382,7 +382,7 @@ fn serialize_variant(
} }
Style::Struct => { Style::Struct => {
let mut pat = builder.pat().struct_().id(type_ident).id(variant_ident).build(); let mut pat = builder.pat().struct_().id(type_ident).id(variant_ident).build();
for field in variant.fields.iter() { for field in &variant.fields {
let name = match field.ident { let name = match field.ident {
Some(name) => name, Some(name) => name,
None => cx.span_bug(field.span, "struct variant has unnamed fields"), None => cx.span_bug(field.span, "struct variant has unnamed fields"),