On 20 Jan 2017, at 12:53, Yann Ylavic <ylavic....@gmail.com> wrote: >> >> apr_hash(sha256_ctx, &buf, &len, plain, plainlen, pool); > > Probably apr_crypto_hash() since apr_hash() is non-crypto hashtable in APR. > Would be nice to have init/update/finish versions too.
I think global rename to apr_digest would be called for (and leave the few existing ones that are crypto hash in place, Dw.