diff options
| author | Flavio Percoco <flaper87@gmail.com> | 2014-04-23 18:16:06 +0200 |
|---|---|---|
| committer | Flavio Percoco <flaper87@gmail.com> | 2014-04-23 18:22:09 +0200 |
| commit | 6e53cfa61e2d63f4dfa628d1b6592b6a729b3172 (patch) | |
| tree | 7609930da13009f3b078a98c2053b7b14e5fd4d9 /src/libsyntax/ext | |
| parent | aff620de1e5e0791b7c91f765cf17f3214848230 (diff) | |
| download | rust-6e53cfa61e2d63f4dfa628d1b6592b6a729b3172.tar.gz rust-6e53cfa61e2d63f4dfa628d1b6592b6a729b3172.zip | |
syntax: fix de-@rooting fallout
Diffstat (limited to 'src/libsyntax/ext')
| -rw-r--r-- | src/libsyntax/ext/deriving/generic.rs | 23 |
1 files changed, 11 insertions, 12 deletions
diff --git a/src/libsyntax/ext/deriving/generic.rs b/src/libsyntax/ext/deriving/generic.rs index 914451fb402..852dd0ed02a 100644 --- a/src/libsyntax/ext/deriving/generic.rs +++ b/src/libsyntax/ext/deriving/generic.rs @@ -814,26 +814,25 @@ impl<'a> MethodDef<'a> { "no self match on an enum in \ generic `deriving`"); } + + // `ref` inside let matches is buggy. Causes havoc wih rusc. + // let (variant_index, ref self_vec) = matches_so_far[0]; + let (variant, self_vec) = match matches_so_far.get(0) { + &(_, v, ref s) => (v, s) + }; + // we currently have a vec of vecs, where each // subvec is the fields of one of the arguments, // but if the variants all match, we want this as // vec of tuples, where each tuple represents a // field. - let substructure; - // most arms don't have matching variants, so do a // quick check to see if they match (even though // this means iterating twice) instead of being // optimistic and doing a pile of allocations etc. - match matching { + let substructure = match matching { Some(variant_index) => { - // `ref` inside let matches is buggy. Causes havoc wih rusc. - // let (variant_index, ref self_vec) = matches_so_far[0]; - let (variant, self_vec) = match matches_so_far.get(0) { - &(_, v, ref s) => (v, s) - }; - let mut enum_matching_fields = Vec::from_elem(self_vec.len(), Vec::new()); for triple in matches_so_far.tail().iter() { @@ -856,12 +855,12 @@ impl<'a> MethodDef<'a> { other: (*other).clone() } }).collect(); - substructure = EnumMatching(variant_index, variant, field_tuples); + EnumMatching(variant_index, variant, field_tuples) } None => { - substructure = EnumNonMatching(matches_so_far.as_slice()); + EnumNonMatching(matches_so_far.as_slice()) } - } + }; self.call_substructure_method(cx, trait_, type_ident, self_args, nonself_args, &substructure) |
