subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bert Huijben" <>
Subject RE: svn commit: r1499315 - in /subversion/branches/verify-keep-going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c
Date Wed, 03 Jul 2013 14:36:15 GMT

> -----Original Message-----
> From: Daniel Shahaf []
> Sent: woensdag 3 juli 2013 13:38
> To:
> Cc:;
> Subject: Re: svn commit: r1499315 - in /subversion/branches/verify-keep-
> going/subversion: libsvn_repos/dump.c svnadmin/svnadmin.c
> 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?

No, not in the existing function. That is a breaking change to api users.

UI clients usually don't want to show things in the same way as 'svn'. They
might want to show it in different ways or want to include only parts of the
error if they recognize the error code. And if you go this way this is
impossible due to localization.

This sort of formatting belongs in svn_cmdline if it is shared between
commandline clients, not in the lower layers of the error api.

I don't see why '_("E%06d: %s"),' should be localized. I don't think it
really can be with just this string.


View raw message