JuliaCrypto / Nettle.jl

Julia wrapper around nettle cryptographic hashing/encryption library providing MD5, SHA1, SHA2 hashing and HMAC functionality, as well as AES encryption/decryption

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove AbstractString names for Ciphers?

nandoconde opened this issue · comments

Revising the code I noticed that names are saved as AbstractString. Not that it matters much performance-wise, since everything is done in the C-callee, but maybe it is unnecessary, and they could be plain old regular String.

Plus, maybe it makes sense to have them be Symbol because it is more idiomatic, but I do not feel very strong about this.

@staticfloat

Agreed, it should be a String, not an AbstractString.