New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add fnv to stdx #38
Add fnv to stdx #38
Conversation
README.md
Outdated
@@ -13,6 +13,7 @@ Current revision: `stdx` 0.118.0-rc, for Rust 1.18, June 8, 2017. | |||
| Date and time | [`chrono = "0.3.1"`] | [📖][d-chrono] | | |||
| Command-line argument parsing | [`clap = "2.24.2"`] | [📖][d-clap] | | |||
| Error handling | [`error-chain = "0.10.0"`] | [📖][d-error-chain] | | |||
| FNV hash function | [`fnv = "1.0.3"`] | [📖][d-fnv] | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since most people won't know what "FNV" is, how about we say "Fast (FN) hashing".
examples/fnv.rs
Outdated
map.remove(&(2)); | ||
println!("The length of HashMap is {}.", map.len()); | ||
println!("The first element is {}.", map.get(&(1)).unwrap()); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The inside of this function needs to be intended one level, and likewise in the markdown.
README.md
Outdated
@@ -1140,6 +1175,7 @@ copyright is owned by its contributors. | |||
[d-clap]: https://docs.rs/clap/2.24.2/clap/ | |||
[d-error-chain]: https://docs.rs/error-chain/0.8.1/error_chain/ | |||
[d-flate2]: https://docs.rs/flate2/0.2.19/flate2/ | |||
[d-fnv]: https://doc.servo.org/fnv/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please make this link "https://docs.rs/fnv/1.0.3/fnv/". All docs on this page link to docs.rs
Ping @alisha17 |
f15dbb3
to
da0a93d
Compare
Thanks @alisha17! |
fnv crate example using FNV HashMap
Fix to issue #34