I would love to see little headers telling how many inserts, deletes and removes there are. That can be a separate change, or you can fold it into this change, but I think it would be awesome.
We should prevent saving models when we don't need to (as this will temporarily lock the table), so before setting and saving, let's check each of those fields to see if anything changed.
I know it's a bit redundant, but I'd rather these made their own queries. The reason is that if a caller is only interested in one of these counts, we shouldn't be performing three aggregate queries. Just slightly more efficient, even if it's a little bit of code duplication. You could potentially create a private helper function in that class for generating a query given a field name, and use that in these functions and in line_counts.
Add Add insert/replace/remove line count to DiffSet and FileDiff.
Review Request #1702 — Created July 6, 2010 and discarded
This patch adds the features requested by Mike Conley for his statistics extension. I was wondering if it wouldn't be useful to change the description on the summary at the beginning and end of each diff viewer to also present this information instead of just the number of chunks. What do you think?
Testing on the command line.