apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Branko ─îibej <br...@xbc.nu>
Subject Re: cvs commit: apr-util/crypto apr_md5.c
Date Wed, 22 Jan 2003 03:18:32 GMT
Justin Erenkrantz wrote:

> --On Tuesday, January 21, 2003 8:27 PM +0000 gstein@apache.org wrote:
>
>>   @@ -145,7 +145,7 @@
>>     * @param inputLen The length of the next message block
>>     */
>>    APU_DECLARE(apr_status_t) apr_md5_update(apr_md5_ctx_t *context,
>> -                                         const unsigned char 
>
> *input,
>
>> +                                         const void *input,
>>                                          apr_size_t inputLen);
>>    /**
>
>
> My hunch is that this breaks binary compatibility.  Do we need to
> bump a version number or do some magic to make our binary compat
> people happy?  -- justin
>
It shouldn't. It could only break binary compatibility on systems where
the representation and/or size of a void* is different than the
representation of a char*, and I bet APR doesn't work on such systems.

-- 
Brane ─îibej   <brane@xbc.nu>   http://www.xbc.nu/brane/


Mime
View raw message