Re: [PATCH] Fix wrong string in subversion/libsvn_repos/reporter.c

2007-12-28 09:49:24 Karl Fogel napisał(a):
> Arfrever Frehtes Taifersar Arahesis <arfrever.fta@gmail.com> writes:
> > This 'fmt' isn't necessary.
> > What do you think about this version?:
>
> I like the new version fine, thanks. To answer your other mail, the
> explanatory comment should go in the code, not the log message.
>
> -Karl
>
> > 2007-12-27 20:01:10 Arfrever Frehtes Taifersar Arahesis napisał(a):
> >> Index: subversion/libsvn_repos/reporter.c
> >> ===================================================================
> >> --- subversion/libsvn_repos/reporter.c (wersja 28655)
> >> +++ subversion/libsvn_repos/reporter.c (kopia robocza)
> >> @@ -165,8 +165,11 @@
> >> if (len + 1 < len)
> >> {
> >> return svn_error_createf(SVN_ERR_REPOS_BAD_REVISION_REPORT, NULL,
> >> - _("Invalid length (%" APR_UINT64_T_FMT ") "
> >> - "when about to read a string"), len);
> >> + apr_psprintf(pool,
> >> + _("Invalid length (%%%s) when "
> >> + "about to read a string"),
> >> + APR_UINT64_T_FMT),
> >> + len);
> >> }
> >>
> >> buf = apr_palloc(pool, len + 1);
> >>

Committed in r28692.

-- 
Arfrever Frehtes Taifersar Arahesis