Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

WIP: Refactor crypto #4346

Closed
wants to merge 6 commits into
from

Conversation

Projects
None yet
4 participants
Owner

indutny commented Dec 2, 2012

Crypto refactor.... still in progress.

Owner

indutny commented Dec 2, 2012

@bnoordhuis it would be probably better to pull it ASAP if it looks good to you, to avoid merge conflicts later.

Member

bnoordhuis commented Dec 3, 2012

What's the motivation for this PR? We're working towards v0.10. Now is not a good time for big changes.

Owner

indutny commented Dec 3, 2012

Well, it's not a big change. I've just shuffled code between header and .cc file. I think it's pretty good time to do it, we can revert Cipher/Decipher merge if you wish to, but all other code changes are harmless.

isaacs commented Jan 17, 2013

Is this still relevant?

Owner

indutny commented Jan 17, 2013

Lets keep it as a reminder for 0.11

Can one of the admins verify this patch?

@indutny indutny closed this Mar 19, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment