diff options
| author | Jonathan Corbet <corbet@lwn.net> | 2015-03-26 13:33:57 -0600 |
|---|---|---|
| committer | Jonathan Corbet <corbet@lwn.net> | 2015-03-26 13:33:57 -0600 |
| commit | 5516be6e392b210a32d95f61cda0f50a8bcf5be4 (patch) | |
| tree | 804dfab69fa888c9afa1ccc5fde184e00735cb9e /database.py | |
| parent | 5e7f0d50fb869b0af4e3f2c9c7b3ebf04ebe4adb (diff) | |
Make "lines changed" accounting consistent
We were calculating "total lines changed" on a per-patch basis, while a
developer's lines changed were max(added,removed) for the entire period
under study. Track developer changed lines per-patch as well...this has
the nice effect of making the percentages add up to 100...
Diffstat (limited to 'database.py')
| -rw-r--r-- | database.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/database.py b/database.py index ca48345..cb242c1 100644 --- a/database.py +++ b/database.py @@ -18,7 +18,7 @@ class Hacker: self.id = id self.employer = [ elist ] self.email = [ email ] - self.added = self.removed = 0 + self.changed = self.added = self.removed = 0 self.patches = [ ] self.signoffs = [ ] self.reviews = [ ] @@ -44,6 +44,7 @@ class Hacker: def addpatch (self, patch): self.added += patch.added self.removed += patch.removed + self.changed += max(patch.added, patch.removed) self.patches.append (patch) # |