On Tue, Jan 21, 2003 at 12:19:37PM -0800, Blair Zajac wrote: > [EMAIL PROTECTED] wrote: > > Blair Zajac <[EMAIL PROTECTED]> writes: > > > [EMAIL PROTECTED] wrote: >... > > > > I disagree. This is a collection of bytes, treated as bytes, not as > > > > characters. I believe APR's typing choice to be a) more accurate, and > > > > b) more revealing about the usage for the data. 'char *' looks like > > > > something that's human-readable -- these digest are not. > > > > > > Why not use void * then? > > > > Why bother? 'unsigned char' is, as far as I know, the way to > > represent 8 bits of data. This has been the case in every piece of > > software I've ever worked on. > > > > You wanna change APR? > > No, definitely don't want to go there. :)
Hunh? It isn't like APR is a minefield. Sheesh. I've updated apr-util/include/apr_md5.h and crypto/apr_md5.c. The prototype now takes a "const void *" for apr_md5_update() and apr_md5(). Cheers, -g -- Greg Stein, http://www.lyra.org/
