Re: useless return values for apr_md5_*() functions...

-1 (not a veto)

With the release of a major application (Apache 2.0), the APR API should not change unless
there is a compelling reason. This is not a compelling reason.

Bill

> I'm planning to make the following functions return void instead of
> apr_status_t:
>
> apr_md5_init()
> apr_md5_update()
> apr_md5_final()
> apr_md5_encode()
> apr_md5() /* this one will get the obvious code tweak too */
>
> They currently can only return success anyway, and it's hard to
> conceive of an implementation that would do otherwise.
>
> If any APR developer knows a reason not to make this change, please
> speak now or forever hold your peace... :-) (I'll wait a day.)
>
> -Karl
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 12 22:28:08 2002