Re: [PATCH] Calculate lines changed for cvs log command

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Fredrik Noring
Date: Sunday, April 13, 2008 - 11:44 am

Hi Jakub,

OK, I'll look deeper into the mailer issues. Any comments on the  
actual code in the patch, and the question regarding git-log vs git- 
cat-file?

Speaking of which -- is there any particular reason for not running  
git-log once, instead of forking it (or git-cat-file) for every commit  
in the log? There seems to be a special case for merges, but it'd  
appear to be more efficient to query the SQLite DB for the commits  
provided by git-log rather than the other way around, no?

Any thoughts?

(git-send-email appears to fail for me without providing an error.)

Many thanks,
Fredrik

13 apr 2008 kl. 18.59 skrev Jakub Narebski:

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Calculate lines changed for cvs log command, Fredrik Noring, (Sun Apr 13, 8:27 am)
Re: [PATCH] Calculate lines changed for cvs log command, Jakub Narebski, (Sun Apr 13, 9:59 am)
Re: [PATCH] Calculate lines changed for cvs log command, Fredrik Noring, (Sun Apr 13, 11:44 am)
Re: [PATCH] Calculate lines changed for cvs log command, Jakub Narebski, (Sun Apr 13, 1:43 pm)
Re: [PATCH] Calculate lines changed for cvs log command, Fredrik Noring, (Mon Apr 14, 12:56 pm)