Re: misaligned blame output if repo has >1m revisions

On Fri, Apr 9, 2010 at 9:17 PM, Johan Corveleyn <jcorvel_at_gmail.com> wrote:

> Hi devs,
>
> Just a heads up, not really important, but I came across this in
> blame_cmd.c (inside print_line_info):
>
> [[[
> const char *rev_str = SVN_IS_VALID_REVNUM(revision)
> ? apr_psprintf(pool, "%6ld", revision)
> : " -";
>
> ]]]
>
> Which means that users of a repo with over 1 million revisions will
> see misaligned blame output ...
>
> So I guess this is coming up for you guys when s.a.o reaches the 1
> million mark :-).
>

Heh. We also have a similar problem when committer names are longer than a
given threshold. I've thought about hacking a solution into the client, but
have never really gotten around to it. Do you want to write a patch?

-Hyrum
Received on 2010-04-09 22:38:27 CEST