You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Hudson <gh...@MIT.EDU> on 2005/10/27 05:47:40 UTC
Re: svn commit: r17034 - in trunk/subversion: mod_authz_svn
mod_dav_svn
On Wed, 2005-10-26 at 11:02 -0500, kfogel@tigris.org wrote:
> + (svn_err->message ?
> + svn_err->message :
> + "[no further information available]"));
A message field of NULL is supposed to mean "use the description of the
error code", not "use a placeholder string". Perhaps we could add a
helper function to libsvn_subr.c to simplify these cases.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: svn commit: r17034 - in trunk/subversion: mod_authz_svn mod_dav_svn
Posted by kf...@collab.net.
Greg Hudson <gh...@MIT.EDU> writes:
> On Wed, 2005-10-26 at 11:02 -0500, kfogel@tigris.org wrote:
> > + (svn_err->message ?
> > + svn_err->message :
> > + "[no further information available]"));
>
> A message field of NULL is supposed to mean "use the description of the
> error code", not "use a placeholder string". Perhaps we could add a
> helper function to libsvn_subr.c to simplify these cases.
Just FYI, I'm doing this, will commit as soon as can.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org