2009/6/27 Jakub Narebski <jnareb@xxxxxxxxx>:
> By the way, this "atnight" warning is duplicated in subroutine below
> git_print_authorship_rows(), but IMHO it is not something very important.
> It can be left for later.
I'll resend this patch, with this thing refactored too, and with the
localtime fix.
>> + foreach my $who (@people) {
>> + my %ad = parse_date($co->{$who . '_epoch'}, $co->{$who . '_tz'});
>
> Wouldn't
>
> $co->{"${who}_epoch"}
>
> be simpler than
>
> $co->{$who . '_epoch'}
>
> Also it is now %wd (%who_date) or just %date rather than
> %ad (%author_date) vs %cd (%committer_date).
>
>
> Both of those issues are very, very minor.
But I can tune them out 8-)
> Nice... although this set of diff chunks made me wonder why we
> use "10" for 'shortlog', "15, 3" for 'history, and "15, 5" for
> 'search' (grep).
I don't know about that. It might be worth considering unifying them.
> Thanks again for your diligent work on this patch series!
Thank you very much for your comments. I could clearly see the code
improving significantly at each iteration. It's a great sastisfaction,
even though it's quite some work.
--
Giuseppe "Oblomov" Bilotta
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html