[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Greg Hudson <ghudson_at_MIT.EDU>
Date: 2005-03-11 19:44:29 CET

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.