Skip to content

Commit dee8423

Browse files
committed
auto merge of #16768 : nham/rust/libcollections_test_cleanup, r=alexcrichton
unused imports. This is mostly converting uses of `push_back`, `pop_back`, `shift` and `unshift` to `push`, `pop`, `remove` and `insert`.
2 parents 2e92c67 + 7b31058 commit dee8423

File tree

4 files changed

+75
-77
lines changed

4 files changed

+75
-77
lines changed

src/libcollections/dlist.rs

Lines changed: 30 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -805,21 +805,21 @@ mod tests {
805805
fn test_basic() {
806806
let mut m: DList<Box<int>> = DList::new();
807807
assert_eq!(m.pop_front(), None);
808-
assert_eq!(m.pop_back(), None);
808+
assert_eq!(m.pop(), None);
809809
assert_eq!(m.pop_front(), None);
810810
m.push_front(box 1);
811811
assert_eq!(m.pop_front(), Some(box 1));
812-
m.push_back(box 2);
813-
m.push_back(box 3);
812+
m.push(box 2);
813+
m.push(box 3);
814814
assert_eq!(m.len(), 2);
815815
assert_eq!(m.pop_front(), Some(box 2));
816816
assert_eq!(m.pop_front(), Some(box 3));
817817
assert_eq!(m.len(), 0);
818818
assert_eq!(m.pop_front(), None);
819-
m.push_back(box 1);
820-
m.push_back(box 3);
821-
m.push_back(box 5);
822-
m.push_back(box 7);
819+
m.push(box 1);
820+
m.push(box 3);
821+
m.push(box 5);
822+
m.push(box 7);
823823
assert_eq!(m.pop_front(), Some(box 1));
824824

825825
let mut n = DList::new();
@@ -856,19 +856,19 @@ mod tests {
856856
{
857857
let mut m = DList::new();
858858
let mut n = DList::new();
859-
n.push_back(2i);
859+
n.push(2i);
860860
m.append(n);
861861
assert_eq!(m.len(), 1);
862-
assert_eq!(m.pop_back(), Some(2));
862+
assert_eq!(m.pop(), Some(2));
863863
check_links(&m);
864864
}
865865
{
866866
let mut m = DList::new();
867867
let n = DList::new();
868-
m.push_back(2i);
868+
m.push(2i);
869869
m.append(n);
870870
assert_eq!(m.len(), 1);
871-
assert_eq!(m.pop_back(), Some(2));
871+
assert_eq!(m.pop(), Some(2));
872872
check_links(&m);
873873
}
874874

@@ -889,10 +889,10 @@ mod tests {
889889
{
890890
let mut m = DList::new();
891891
let mut n = DList::new();
892-
n.push_back(2i);
892+
n.push(2i);
893893
m.prepend(n);
894894
assert_eq!(m.len(), 1);
895-
assert_eq!(m.pop_back(), Some(2));
895+
assert_eq!(m.pop(), Some(2));
896896
check_links(&m);
897897
}
898898

@@ -950,9 +950,9 @@ mod tests {
950950
#[test]
951951
fn test_iterator_clone() {
952952
let mut n = DList::new();
953-
n.push_back(2i);
954-
n.push_back(3);
955-
n.push_back(4);
953+
n.push(2i);
954+
n.push(3);
955+
n.push(4);
956956
let mut it = n.iter();
957957
it.next();
958958
let mut jt = it.clone();
@@ -1007,7 +1007,7 @@ mod tests {
10071007
let mut n = DList::new();
10081008
assert!(n.mut_iter().next().is_none());
10091009
n.push_front(4i);
1010-
n.push_back(5);
1010+
n.push(5);
10111011
let mut it = n.mut_iter();
10121012
assert_eq!(it.size_hint(), (2, Some(2)));
10131013
assert!(it.next().is_some());
@@ -1081,8 +1081,8 @@ mod tests {
10811081
assert_eq!(n.pop_front(), Some(1));
10821082

10831083
let mut m = DList::new();
1084-
m.push_back(2i);
1085-
m.push_back(4);
1084+
m.push(2i);
1085+
m.push(4);
10861086
m.insert_ordered(3);
10871087
check_links(&m);
10881088
assert_eq!(vec![2,3,4], m.move_iter().collect::<Vec<int>>());
@@ -1119,7 +1119,7 @@ mod tests {
11191119
assert!(n == m);
11201120
n.push_front(1);
11211121
assert!(n != m);
1122-
m.push_back(1);
1122+
m.push(1);
11231123
assert!(n == m);
11241124

11251125
let n = list_from([2i,3,4]);
@@ -1134,9 +1134,9 @@ mod tests {
11341134

11351135
assert!(hash::hash(&x) == hash::hash(&y));
11361136

1137-
x.push_back(1i);
1138-
x.push_back(2);
1139-
x.push_back(3);
1137+
x.push(1i);
1138+
x.push(2);
1139+
x.push(3);
11401140

11411141
y.push_front(3i);
11421142
y.push_front(2);
@@ -1216,19 +1216,19 @@ mod tests {
12161216
let r: u8 = rand::random();
12171217
match r % 6 {
12181218
0 => {
1219-
m.pop_back();
1219+
m.pop();
12201220
v.pop();
12211221
}
12221222
1 => {
12231223
m.pop_front();
1224-
v.shift();
1224+
v.remove(0);
12251225
}
12261226
2 | 4 => {
12271227
m.push_front(-i);
1228-
v.unshift(-i);
1228+
v.insert(0, -i);
12291229
}
12301230
3 | 5 | _ => {
1231-
m.push_back(i);
1231+
m.push(i);
12321232
v.push(i);
12331233
}
12341234
}
@@ -1264,16 +1264,16 @@ mod tests {
12641264
fn bench_push_back(b: &mut test::Bencher) {
12651265
let mut m: DList<int> = DList::new();
12661266
b.iter(|| {
1267-
m.push_back(0);
1267+
m.push(0);
12681268
})
12691269
}
12701270

12711271
#[bench]
12721272
fn bench_push_back_pop_back(b: &mut test::Bencher) {
12731273
let mut m: DList<int> = DList::new();
12741274
b.iter(|| {
1275-
m.push_back(0);
1276-
m.pop_back();
1275+
m.push(0);
1276+
m.pop();
12771277
})
12781278
}
12791279

src/libcollections/hash/mod.rs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,6 @@ impl<S: Writer, T: Hash<S>, U: Hash<S>> Hash<S> for Result<T, U> {
293293

294294
#[cfg(test)]
295295
mod tests {
296-
use std::prelude::*;
297296
use std::mem;
298297

299298
use slice::ImmutableSlice;

0 commit comments

Comments
 (0)