Skip to content
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

Split up salsa20-family crate into salsa20-core, salsa20 and chacha20 #16

Merged
merged 5 commits into from Jun 24, 2019

Conversation

sseemayer
Copy link
Contributor

@sseemayer sseemayer commented Jun 23, 2019

This is a PR to split up salsa20-family into salsa20-core, salsa20 and chacha20 as discussed in #15 .

  • Fix all the broken references between crates
  • Update README

@sseemayer
Copy link
Contributor Author

sseemayer commented Jun 23, 2019

Apologies about the larger-than necessary diffs, especially 9e52d49 . I have set up my editor to automatically run rustfmt on every save and didn't catch this quickly enough.

chacha20/Cargo.toml Outdated Show resolved Hide resolved
@sseemayer sseemayer marked this pull request as ready for review June 23, 2019 18:39
@newpavlov
Copy link
Member

At the first glance looks good to me!

@tarcieri
Will you handle a more in-depth review, merging and publishing of those crates?

@tarcieri
Copy link
Member

Yep, I can look later today.

@tarcieri
Copy link
Member

Looks good to me as well. Thanks for the help!

@emc2 this would be good for you to (retroactively) review as well, if you have time.

@tarcieri tarcieri merged commit e455aa6 into RustCrypto:master Jun 24, 2019
@sseemayer sseemayer deleted the split-salsa20-family branch June 28, 2019 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants