Re: svn commit: r13370 - in branches/locking/subversion: include libsvn_client libsvn_ra_dav libsvn_ra_local libsvn_ra_svn libsvn_subr

On Fri, 2005-03-11 at 11:41, fitz@tigris.org wrote:
> - * path, FALSE otherwise. */
> + * path, FALSE otherwise. @a err may not be NULL. */
> svn_boolean_t svn_error_is_lock_error (svn_error_t *err);

I'm not very fond of this precedent at all. Someone reading this
docstring might assume that other functions accept NULL values of any
pointer parameter because they don't specify either way. Whereas I
would would say that a function must document that it *does* accept a
NULL value of a pointer parameter, if it does so.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 11 19:47:58 2005