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

Re: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Wed, 3 Jul 2013 14:37:54 +0300

prabhugs_at_apache.org wrote on Wed, Jul 03, 2013 at 10:49:02 -0000:
> + notify_failure->warning_str = apr_psprintf(pool,
> + _("E%06d: %s"),

Would it make sense to push the E%06d: part down to the implementation
of svn_err_best_message()?

I.e., if the information is useful for this caller, presumably it'll be
useful for others too?

> + err->apr_err,
> + svn_err_best_message(err, errbuf,
> + sizeof(errbuf)));
Received on 2013-07-03 13:38:32 CEST

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.