Re: svn commit: r17625 - trunk/subversion/libsvn_fs_base

On 12/4/05, mbk@tigris.org <mbk@tigris.org> wrote:

> +static svn_error_t *
> +unknown_node_kind (const char *rep_key)
> +{
> + return svn_error_createf
> + (SVN_ERR_FS_CORRUPT, NULL,
> + _("Unknown node kind for representation '%s'"), rep_key);
> +}

Speaking as someone who's spent a lot of time in libsvn_ra_dav lately,
I'm getting more and more unthrilled about these sort of wrapper
functions for generating errors. They make the file and line info in
maintainer mode builds essentially useless, and if we could avoid them
going forward I'm sure that the future developers who have to debug
this code would appreciate it.

-garrett

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Dec 5 01:29:15 2005