Skip to content

Commit cac4891

Browse files
committed
Fixups
1 parent 0700154 commit cac4891

File tree

5 files changed

+31
-27
lines changed

5 files changed

+31
-27
lines changed

src/librustc/util/enum_set.rs

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -207,19 +207,19 @@ mod test {
207207
fn test_each() {
208208
let mut e1: EnumSet<Foo> = EnumSet::empty();
209209

210-
assert_eq!(~[], iter::to_vec(|f| e1.each(f)))
210+
assert_eq!(~[], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e1.each(f)))
211211

212212
e1.add(A);
213-
assert_eq!(~[A], iter::to_vec(|f| e1.each(f)))
213+
assert_eq!(~[A], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e1.each(f)))
214214

215215
e1.add(C);
216-
assert_eq!(~[A,C], iter::to_vec(|f| e1.each(f)))
216+
assert_eq!(~[A,C], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e1.each(f)))
217217

218218
e1.add(C);
219-
assert_eq!(~[A,C], iter::to_vec(|f| e1.each(f)))
219+
assert_eq!(~[A,C], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e1.each(f)))
220220

221221
e1.add(B);
222-
assert_eq!(~[A,B,C], iter::to_vec(|f| e1.each(f)))
222+
assert_eq!(~[A,B,C], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e1.each(f)))
223223
}
224224

225225
///////////////////////////////////////////////////////////////////////////
@@ -236,12 +236,12 @@ mod test {
236236
e2.add(C);
237237

238238
let e_union = e1 | e2;
239-
assert_eq!(~[A,B,C], iter::to_vec(|f| e_union.each(f)))
239+
assert_eq!(~[A,B,C], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e_union.each(f)))
240240

241241
let e_intersection = e1 & e2;
242-
assert_eq!(~[C], iter::to_vec(|f| e_intersection.each(f)))
242+
assert_eq!(~[C], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e_intersection.each(f)))
243243

244244
let e_subtract = e1 - e2;
245-
assert_eq!(~[A], iter::to_vec(|f| e_subtract.each(f)))
245+
assert_eq!(~[A], iter::FromIter::from_iter::<Foo, ~[Foo]>(|f| e_subtract.each(f)))
246246
}
247247
}

src/libstd/iter.rs

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -65,16 +65,6 @@ pub trait FromIter<T> {
6565
pub fn from_iter(iter: &fn(f: &fn(T) -> bool) -> bool) -> Self;
6666
}
6767

68-
// NOTE: This should be in vec but can't because of coherence
69-
impl<T> FromIter<T> for ~[T]{
70-
#[inline(always)]
71-
pub fn from_iter(iter: &fn(f: &fn(T) -> bool) -> bool) -> ~[T] {
72-
let mut v = ~[];
73-
for iter |x| { v.push(x) }
74-
v
75-
}
76-
}
77-
7868
/**
7969
* Return true if `predicate` is true for any values yielded by an internal iterator.
8070
*
@@ -269,7 +259,7 @@ mod tests {
269259

270260
#[test]
271261
fn test_from_iter() {
272-
let xs: = ~[1, 2, 3];
262+
let xs = ~[1, 2, 3];
273263
let ys: ~[int] = do FromIter::from_iter |f| { xs.each(|x| f(*x)) };
274264
assert_eq!(xs, ys);
275265
}

src/libstd/iterator.rs

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,12 @@ implementing the `Iterator` trait.
1919

2020
use cmp;
2121
use iter;
22-
use iter::FromIter;
22+
use iter::{FromIter, Times};
2323
use num::{Zero, One};
24-
use prelude::*;
24+
use option::{Option, Some, None};
25+
use ops::{Add, Mul};
26+
use cmp::Ord;
27+
use clone::Clone;
2528

2629
/// An interface for dealing with "external iterators". These types of iterators
2730
/// can be resumed at any time as all state is stored internally as opposed to
@@ -871,9 +874,9 @@ mod tests {
871874
use uint;
872875

873876
#[test]
874-
fn test_counter_to_vec() {
877+
fn test_counter_from_iter() {
875878
let mut it = Counter::new(0, 5).take(10);
876-
let xs = iter::to_vec(|f| it.advance(f));
879+
let xs: ~[int] = iter::FromIter::from_iter::<int, ~[int]>(|f| it.advance(f));
877880
assert_eq!(xs, ~[0, 5, 10, 15, 20, 25, 30, 35, 40, 45]);
878881
}
879882

@@ -904,7 +907,7 @@ mod tests {
904907
fn test_filter_map() {
905908
let mut it = Counter::new(0u, 1u).take(10)
906909
.filter_map(|x: uint| if x.is_even() { Some(x*x) } else { None });
907-
assert_eq!(it.to_vec(), ~[0*0, 2*2, 4*4, 6*6, 8*8]);
910+
assert_eq!(it.collect::<~[uint]>(), ~[0*0, 2*2, 4*4, 6*6, 8*8]);
908911
}
909912

910913
#[test]
@@ -1065,7 +1068,7 @@ mod tests {
10651068

10661069
#[test]
10671070
fn test_collect() {
1068-
let a = [1, 2, 3, 4, 5];
1071+
let a = ~[1, 2, 3, 4, 5];
10691072
let b: ~[int] = a.iter().transform(|&x| x).collect();
10701073
assert_eq!(a, b);
10711074
}

src/libstd/prelude.rs

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ Many programming languages have a 'prelude': a particular subset of the
1414
libraries that come with the language. Every program imports the prelude by
1515
default.
1616
17-
For example, it would be annoying to add `use io::println;` to every single
17+
For example, it would be annoying to add `use std::io::println;` to every single
1818
program, and the vast majority of Rust programs will wish to print to standard
1919
output. Therefore, it makes sense to import it into every program.
2020
@@ -49,7 +49,8 @@ pub use hash::Hash;
4949
pub use old_iter::{BaseIter, ReverseIter, MutableIter, ExtendedIter, EqIter};
5050
pub use old_iter::{CopyableIter, CopyableOrderedIter, CopyableNonstrictIter};
5151
pub use old_iter::{ExtendedMutableIter};
52-
pub use iter::Times;
52+
pub use iter::{Times, FromIter};
53+
// FIXME: #5898 pub use iterator::{Iterator, IteratorUtil};
5354
pub use num::{Num, NumCast};
5455
pub use num::{Orderable, Signed, Unsigned, Round};
5556
pub use num::{Algebraic, Trigonometric, Exponential, Hyperbolic};

src/libstd/vec.rs

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ use clone::Clone;
2020
use old_iter::BaseIter;
2121
use old_iter;
2222
use iterator::Iterator;
23+
use iter::FromIter;
2324
use kinds::Copy;
2425
use libc;
2526
use old_iter::CopyableIter;
@@ -2996,6 +2997,15 @@ impl<'self, T> Iterator<&'self mut T> for MutVecIterator<'self, T> {
29962997
}
29972998
}
29982999

3000+
impl<T> FromIter<T> for ~[T]{
3001+
#[inline(always)]
3002+
pub fn from_iter(iter: &fn(f: &fn(T) -> bool) -> bool) -> ~[T] {
3003+
let mut v = ~[];
3004+
for iter |x| { v.push(x) }
3005+
v
3006+
}
3007+
}
3008+
29993009
#[cfg(test)]
30003010
mod tests {
30013011
use option::{None, Option, Some};

0 commit comments

Comments
 (0)