From 3709aa78d8fc75ed739e837d7c96205b8494188d Mon Sep 17 00:00:00 2001 From: blake2-ppc Date: Sat, 28 Sep 2013 04:37:20 +0200 Subject: [PATCH] std::vec: Remove functions concat, connect std::vec::{concat, connect, concat_slices, connect_slices} are replaced by the already existing trait methods .concat_vec() and .connect_vec(). --- src/librustc/middle/check_match.rs | 4 ++-- src/librustdoc/clean.rs | 11 +++++------ src/libstd/vec.rs | 25 +++++-------------------- 3 files changed, 12 insertions(+), 28 deletions(-) diff --git a/src/librustc/middle/check_match.rs b/src/librustc/middle/check_match.rs index 8fc98ac10c7be..c9eded645fa2d 100644 --- a/src/librustc/middle/check_match.rs +++ b/src/librustc/middle/check_match.rs @@ -773,12 +773,12 @@ pub fn specialize(cx: &MatchCheckCtxt, let num_elements = before.len() + after.len(); if num_elements < arity && slice.is_some() { Some(vec::append( - vec::concat(&[ + [ before, vec::from_elem( arity - num_elements, wild()), after - ]), + ].concat_vec(), r.tail() )) } else if num_elements == arity { diff --git a/src/librustdoc/clean.rs b/src/librustdoc/clean.rs index 623ac82486d26..70f1815b462ed 100644 --- a/src/librustdoc/clean.rs +++ b/src/librustdoc/clean.rs @@ -173,12 +173,11 @@ impl Clean for doctree::Module { visibility: self.vis.clean(), id: self.id, inner: ModuleItem(Module { - items: std::vec::concat(&[self.structs.clean(), - self.enums.clean(), self.fns.clean(), - std::vec::concat(self.foreigns.clean()), - self.mods.clean(), self.typedefs.clean(), - self.statics.clean(), self.traits.clean(), - self.impls.clean(), self.view_items.clean()]) + items: [self.structs.clean(), self.enums.clean(), + self.fns.clean(), self.foreigns.clean().concat_vec(), + self.mods.clean(), self.typedefs.clean(), + self.statics.clean(), self.traits.clean(), + self.impls.clean(), self.view_items.clean()].concat_vec() }) } } diff --git a/src/libstd/vec.rs b/src/libstd/vec.rs index 4d7d4bffdf148..1398f08bb7c9b 100644 --- a/src/libstd/vec.rs +++ b/src/libstd/vec.rs @@ -340,18 +340,6 @@ pub fn flat_map(v: &[T], f: &fn(t: &T) -> ~[U]) -> ~[U] { result } -/// Flattens a vector of vectors of T into a single vector of T. -pub fn concat(v: &[~[T]]) -> ~[T] { v.concat_vec() } - -/// Concatenate a vector of vectors, placing a given separator between each -pub fn connect(v: &[~[T]], sep: &T) -> ~[T] { v.connect_vec(sep) } - -/// Flattens a vector of vectors of T into a single vector of T. -pub fn concat_slices(v: &[&[T]]) -> ~[T] { v.concat_vec() } - -/// Concatenate a vector of vectors, placing a given separator between each -pub fn connect_slices(v: &[&[T]], sep: &T) -> ~[T] { v.connect_vec(sep) } - #[allow(missing_doc)] pub trait VectorVector { // FIXME #5898: calling these .concat and .connect conflicts with @@ -3098,24 +3086,21 @@ mod tests { #[test] fn test_concat() { - assert_eq!(concat([~[1], ~[2,3]]), ~[1, 2, 3]); + let v: [~[int], ..0] = []; + assert_eq!(v.concat_vec(), ~[]); assert_eq!([~[1], ~[2,3]].concat_vec(), ~[1, 2, 3]); - assert_eq!(concat_slices([&[1], &[2,3]]), ~[1, 2, 3]); assert_eq!([&[1], &[2,3]].concat_vec(), ~[1, 2, 3]); } #[test] fn test_connect() { - assert_eq!(connect([], &0), ~[]); - assert_eq!(connect([~[1], ~[2, 3]], &0), ~[1, 0, 2, 3]); - assert_eq!(connect([~[1], ~[2], ~[3]], &0), ~[1, 0, 2, 0, 3]); + let v: [~[int], ..0] = []; + assert_eq!(v.connect_vec(&0), ~[]); assert_eq!([~[1], ~[2, 3]].connect_vec(&0), ~[1, 0, 2, 3]); assert_eq!([~[1], ~[2], ~[3]].connect_vec(&0), ~[1, 0, 2, 0, 3]); - assert_eq!(connect_slices([], &0), ~[]); - assert_eq!(connect_slices([&[1], &[2, 3]], &0), ~[1, 0, 2, 3]); - assert_eq!(connect_slices([&[1], &[2], &[3]], &0), ~[1, 0, 2, 0, 3]); + assert_eq!(v.connect_vec(&0), ~[]); assert_eq!([&[1], &[2, 3]].connect_vec(&0), ~[1, 0, 2, 3]); assert_eq!([&[1], &[2], &[3]].connect_vec(&0), ~[1, 0, 2, 0, 3]); }