From 13863e64f8d4399ff5d0f3b9cac1d462551aeeee Mon Sep 17 00:00:00 2001 From: Dan Christensen Date: Sun, 2 Feb 2014 23:44:04 -0500 Subject: [PATCH] Correct a theoretical bug in helpers.prune_split --- attic/helpers.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/attic/helpers.py b/attic/helpers.py index 1d9b5ba69..e8a0d8354 100644 --- 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