Correct a theoretical bug in helpers.prune_split

2014-02-02 23:44:04 -05:00 · 2014-02-02 23:44:04 -05:00 · 13863e64f8
parent c45aedf480
commit 13863e64f8
1 changed files with 4 additions and 3 deletions
--- a/attic/helpers.py
+++ b/attic/helpers.py
@ -99,10 +99,11 @@ def prune_split(archives, pattern, n, skip=[]):
        items.setdefault(key, [])
        items[key].append(a)
    for key, values in sorted(items.items(), reverse=True):
-        if n and values[0] not in skip:
+        if n:
            values.sort(key=attrgetter('ts'), reverse=True)
-            keep.append(values[0])
-            n -= 1
+            if values[0] not in skip:
+                keep.append(values[0])
+                n -= 1
    return keep