Draft: Remove the sha512 libsodium include from base.h
This merge requests moves some implementation details to the cpp, such that we no longer need to include sodium/crypto_hash_sha512.h
in the header file.
It now allocates the sha512 context on the heap, instead of the stack. Not calling _SHA512Final
, will result in the context not being cleaned up, and thus a memory leak. But I think that's OK, since the _SHA512Init, and _SHA512Final functions are not part of the public API, and we only use them in one place, were both of them are called. It also wouldn't make sense to call _SHA512Init without calling _SHA512Final, so I don't expect errors with this in the future.
Edited by Hans Harmannij