-
Notifications
You must be signed in to change notification settings - Fork 35
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
MD5 #50
Comments
Hah, we actually did this at the same time (like to the hour I think). I did the "Let MD5.jl depend on SHA.jl" |
Did you look at the performance? I got rid of all branches in |
I did not. I think the most performant (and also cleanest) would be somewhere between the two. |
We can discuss it here JuliaCrypto/MD5.jl#2 |
Just FYI, there is a new JuliaCrypto organization that we're starting to collect packages underneath. |
@oxinabox shall we move MD5 to JuliaCrypto? |
We should, but I can't seem to find my invite. @ViralBShah are you sure you sent the invite to the org right? |
Yeah shows me as pending. Maybe check your spam? Is there a way to see on GitHub itself? |
Ah, if i go to the org page myself I can see it. |
I have a md5 implementation here. It uses the same structure as SHA.jl and I copy pasted some SHA.jl code there. This is not ideal, I can think of a couple of other options:
HashFunctionsBase.jl
package with common codeWhat do you think @staticfloat ?
The text was updated successfully, but these errors were encountered: