Skip to content

Commit e6ac735

Browse files
authored
Merge pull request #114 from Razican/cargo_fmt
cargo fmt
2 parents c820451 + 46c1c96 commit e6ac735

File tree

19 files changed

+990
-692
lines changed

19 files changed

+990
-692
lines changed

benches/bench.rs

Lines changed: 20 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
#![feature(test)]
2-
extern crate test;
3-
extern crate rand;
42
extern crate fnv;
3+
extern crate rand;
4+
extern crate test;
55
#[macro_use]
66
extern crate lazy_static;
77

8-
use std::hash::Hash;
98
use fnv::FnvHasher;
109
use std::hash::BuildHasherDefault;
10+
use std::hash::Hash;
1111
type FnvBuilder = BuildHasherDefault<FnvHasher>;
1212

13-
use test::Bencher;
1413
use test::black_box;
14+
use test::Bencher;
1515

1616
extern crate indexmap;
1717

@@ -21,35 +21,27 @@ use std::collections::HashMap;
2121
use std::iter::FromIterator;
2222

2323
use rand::rngs::SmallRng;
24-
use rand::FromEntropy;
2524
use rand::seq::SliceRandom;
25+
use rand::FromEntropy;
2626

2727
#[bench]
2828
fn new_hashmap(b: &mut Bencher) {
29-
b.iter(|| {
30-
HashMap::<String, String>::new()
31-
});
29+
b.iter(|| HashMap::<String, String>::new());
3230
}
3331

3432
#[bench]
3533
fn new_orderedmap(b: &mut Bencher) {
36-
b.iter(|| {
37-
IndexMap::<String, String>::new()
38-
});
34+
b.iter(|| IndexMap::<String, String>::new());
3935
}
4036

4137
#[bench]
4238
fn with_capacity_10e5_hashmap(b: &mut Bencher) {
43-
b.iter(|| {
44-
HashMap::<String, String>::with_capacity(10_000)
45-
});
39+
b.iter(|| HashMap::<String, String>::with_capacity(10_000));
4640
}
4741

4842
#[bench]
4943
fn with_capacity_10e5_orderedmap(b: &mut Bencher) {
50-
b.iter(|| {
51-
IndexMap::<String, String>::with_capacity(10_000)
52-
});
44+
b.iter(|| IndexMap::<String, String>::with_capacity(10_000));
5345
}
5446

5547
#[bench]
@@ -228,35 +220,31 @@ fn entry_orderedmap_150(b: &mut Bencher) {
228220
fn iter_sum_hashmap_10_000(b: &mut Bencher) {
229221
let c = 10_000;
230222
let mut map = HashMap::with_capacity(c);
231-
let len = c - c/10;
223+
let len = c - c / 10;
232224
for x in 0..len {
233225
map.insert(x, ());
234226
}
235227
assert_eq!(map.len(), len);
236-
b.iter(|| {
237-
map.keys().sum::<usize>()
238-
});
228+
b.iter(|| map.keys().sum::<usize>());
239229
}
240230

241231
#[bench]
242232
fn iter_sum_orderedmap_10_000(b: &mut Bencher) {
243233
let c = 10_000;
244234
let mut map = IndexMap::with_capacity(c);
245-
let len = c - c/10;
235+
let len = c - c / 10;
246236
for x in 0..len {
247237
map.insert(x, ());
248238
}
249239
assert_eq!(map.len(), len);
250-
b.iter(|| {
251-
map.keys().sum::<usize>()
252-
});
240+
b.iter(|| map.keys().sum::<usize>());
253241
}
254242

255243
#[bench]
256244
fn iter_black_box_hashmap_10_000(b: &mut Bencher) {
257245
let c = 10_000;
258246
let mut map = HashMap::with_capacity(c);
259-
let len = c - c/10;
247+
let len = c - c / 10;
260248
for x in 0..len {
261249
map.insert(x, ());
262250
}
@@ -272,7 +260,7 @@ fn iter_black_box_hashmap_10_000(b: &mut Bencher) {
272260
fn iter_black_box_orderedmap_10_000(b: &mut Bencher) {
273261
let c = 10_000;
274262
let mut map = IndexMap::with_capacity(c);
275-
let len = c - c/10;
263+
let len = c - c / 10;
276264
for x in 0..len {
277265
map.insert(x, ());
278266
}
@@ -285,7 +273,8 @@ fn iter_black_box_orderedmap_10_000(b: &mut Bencher) {
285273
}
286274

287275
fn shuffled_keys<I>(iter: I) -> Vec<I::Item>
288-
where I: IntoIterator
276+
where
277+
I: IntoIterator,
289278
{
290279
let mut v = Vec::from_iter(iter);
291280
let mut rng = SmallRng::from_entropy();
@@ -366,12 +355,9 @@ const LOOKUP_MAP_SIZE: u32 = 100_000_u32;
366355
const LOOKUP_SAMPLE_SIZE: u32 = 5000;
367356
const SORT_MAP_SIZE: usize = 10_000;
368357

369-
370358
// use lazy_static so that comparison benchmarks use the exact same inputs
371359
lazy_static! {
372-
static ref KEYS: Vec<u32> = {
373-
shuffled_keys(0..LOOKUP_MAP_SIZE)
374-
};
360+
static ref KEYS: Vec<u32> = { shuffled_keys(0..LOOKUP_MAP_SIZE) };
375361
}
376362

377363
lazy_static! {
@@ -429,7 +415,6 @@ fn lookup_hashmap_100_000_multi(b: &mut Bencher) {
429415
});
430416
}
431417

432-
433418
#[bench]
434419
fn lookup_ordermap_100_000_multi(b: &mut Bencher) {
435420
let map = &*OMAP_100K;
@@ -456,7 +441,6 @@ fn lookup_hashmap_100_000_inorder_multi(b: &mut Bencher) {
456441
});
457442
}
458443

459-
460444
#[bench]
461445
fn lookup_ordermap_100_000_inorder_multi(b: &mut Bencher) {
462446
let map = &*OMAP_100K;
@@ -480,7 +464,6 @@ fn lookup_hashmap_100_000_single(b: &mut Bencher) {
480464
});
481465
}
482466

483-
484467
#[bench]
485468
fn lookup_ordermap_100_000_single(b: &mut Bencher) {
486469
let map = &*OMAP_100K;
@@ -517,7 +500,6 @@ fn grow_fnv_ordermap_100_000(b: &mut Bencher) {
517500
});
518501
}
519502

520-
521503
const MERGE: u64 = 10_000;
522504
#[bench]
523505
fn hashmap_merge_simple(b: &mut Bencher) {
@@ -708,15 +690,13 @@ fn many_retain_hashmap_100_000(b: &mut Bencher) {
708690
});
709691
}
710692

711-
712693
// simple sort impl for comparison
713694
pub fn simple_sort<K: Ord + Hash, V>(m: &mut IndexMap<K, V>) {
714695
let mut ordered: Vec<_> = m.drain(..).collect();
715696
ordered.sort_by(|left, right| left.0.cmp(&right.0));
716697
m.extend(ordered);
717698
}
718699

719-
720700
#[bench]
721701
fn ordermap_sort_s(b: &mut Bencher) {
722702
let map = OMAP_SORT_S.clone();
@@ -770,17 +750,12 @@ fn ordermap_simple_sort_u32(b: &mut Bencher) {
770750
fn ordermap_clone_for_sort_s(b: &mut Bencher) {
771751
let map = OMAP_SORT_S.clone();
772752

773-
b.iter(|| {
774-
map.clone()
775-
});
753+
b.iter(|| map.clone());
776754
}
777755

778756
#[bench]
779757
fn ordermap_clone_for_sort_u32(b: &mut Bencher) {
780758
let map = OMAP_SORT_U32.clone();
781759

782-
b.iter(|| {
783-
map.clone()
784-
});
760+
b.iter(|| map.clone());
785761
}
786-

benches/faststring.rs

Lines changed: 18 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#![feature(test)]
2-
extern crate test;
3-
extern crate rand;
42
extern crate lazy_static;
3+
extern crate rand;
4+
extern crate test;
55

66
use test::Bencher;
77

@@ -12,8 +12,8 @@ use indexmap::IndexMap;
1212
use std::collections::HashMap;
1313
use std::iter::FromIterator;
1414

15-
use rand::thread_rng;
1615
use rand::seq::SliceRandom;
16+
use rand::thread_rng;
1717

1818
use std::hash::{Hash, Hasher};
1919

@@ -24,57 +24,51 @@ use std::ops::Deref;
2424
#[repr(transparent)]
2525
pub struct OneShot<T: ?Sized>(pub T);
2626

27-
impl Hash for OneShot<str>
28-
{
27+
impl Hash for OneShot<str> {
2928
fn hash<H: Hasher>(&self, h: &mut H) {
3029
h.write(self.0.as_bytes())
3130
}
3231
}
3332

3433
impl<'a, S> From<&'a S> for &'a OneShot<str>
35-
where S: AsRef<str>
34+
where
35+
S: AsRef<str>,
3636
{
3737
fn from(s: &'a S) -> Self {
3838
let s: &str = s.as_ref();
39-
unsafe {
40-
&*(s as *const str as *const OneShot<str>)
41-
}
39+
unsafe { &*(s as *const str as *const OneShot<str>) }
4240
}
4341
}
4442

45-
impl Hash for OneShot<String>
46-
{
43+
impl Hash for OneShot<String> {
4744
fn hash<H: Hasher>(&self, h: &mut H) {
4845
h.write(self.0.as_bytes())
4946
}
5047
}
5148

52-
impl Borrow<OneShot<str>> for OneShot<String>
53-
{
49+
impl Borrow<OneShot<str>> for OneShot<String> {
5450
fn borrow(&self) -> &OneShot<str> {
5551
<&OneShot<str>>::from(&self.0)
5652
}
5753
}
5854

59-
impl<T> Deref for OneShot<T>
60-
{
55+
impl<T> Deref for OneShot<T> {
6156
type Target = T;
6257
fn deref(&self) -> &T {
6358
&self.0
6459
}
6560
}
6661

67-
6862
fn shuffled_keys<I>(iter: I) -> Vec<I::Item>
69-
where I: IntoIterator
63+
where
64+
I: IntoIterator,
7065
{
7166
let mut v = Vec::from_iter(iter);
7267
let mut rng = thread_rng();
7368
v.shuffle(&mut rng);
7469
v
7570
}
7671

77-
7872
#[bench]
7973
fn insert_hashmap_string_10_000(b: &mut Bencher) {
8074
let c = 10_000;
@@ -137,7 +131,9 @@ fn lookup_hashmap_10_000_exist_string_oneshot(b: &mut Bencher) {
137131
for &key in &keys {
138132
map.insert(OneShot(key.to_string()), 1);
139133
}
140-
let lookups = (5000..c).map(|x| OneShot(x.to_string())).collect::<Vec<_>>();
134+
let lookups = (5000..c)
135+
.map(|x| OneShot(x.to_string()))
136+
.collect::<Vec<_>>();
141137
b.iter(|| {
142138
let mut found = 0;
143139
for key in &lookups {
@@ -173,7 +169,9 @@ fn lookup_ordermap_10_000_exist_string_oneshot(b: &mut Bencher) {
173169
for &key in &keys {
174170
map.insert(OneShot(key.to_string()), 1);
175171
}
176-
let lookups = (5000..c).map(|x| OneShot(x.to_string())).collect::<Vec<_>>();
172+
let lookups = (5000..c)
173+
.map(|x| OneShot(x.to_string()))
174+
.collect::<Vec<_>>();
177175
b.iter(|| {
178176
let mut found = 0;
179177
for key in &lookups {

src/equivalent.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
1-
21
use std::borrow::Borrow;
32

43
/// Key equivalence trait.
@@ -17,8 +16,9 @@ pub trait Equivalent<K: ?Sized> {
1716
}
1817

1918
impl<Q: ?Sized, K: ?Sized> Equivalent<K> for Q
20-
where Q: Eq,
21-
K: Borrow<Q>,
19+
where
20+
Q: Eq,
21+
K: Borrow<Q>,
2222
{
2323
#[inline]
2424
fn equivalent(&self, key: &K) -> bool {

0 commit comments

Comments
 (0)