On Mon, Nov 15, 2004 at 11:09:40PM +0000, Julian Foad wrote:
> Remove unnecessary type casts that were casting away "const".
> No functional change.

These ones aren't unnecessary, some compilers are more picky about
implicit signed char * -> unsigned char * conversion, so this will
introduce warnings (or possibly errors) with some compilers.

> * apr-util/crypto/apr_md5.c
>   (apr_md5_encode): Remove some type casts.
> 
> Index: apr-util/crypto/apr_md5.c
> ===================================================================
> --- apr-util/crypto/apr_md5.c (revision 65585)
> +++ apr-util/crypto/apr_md5.c (working copy)
> @@ -536,25 +536,25 @@ APU_DECLARE(apr_status_t) apr_md5_encode
>      /*
>       * The password first, since that is what is most unknown
>       */
> -    apr_md5_update(&ctx, (unsigned char *)pw, strlen(pw));
> +    apr_md5_update(&ctx, pw, strlen(pw));
...

Reply via email to