From d9a52572550c1081c3df8a900b0c5545837f3e67 Mon Sep 17 00:00:00 2001 From: Ewoud Kohl van Wijngaarden Date: Fri, 12 Oct 2012 13:00:38 +0200 Subject: [PATCH] Use collections.Counter instead of manually --- find_reduced.py | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/find_reduced.py b/find_reduced.py index 723a9fd..a363d68 100755 --- a/find_reduced.py +++ b/find_reduced.py @@ -8,7 +8,7 @@ if len(sys.argv) != 2: import os from os import path -from collections import defaultdict +from collections import Counter, defaultdict from subprocess import Popen, PIPE from functools import cmp_to_key from datetime import datetime @@ -208,12 +208,7 @@ for d in debian_names: sys.stderr.write("%s\n%s\n"%(list(set(missing_deb_mappings)),list(set(missing_gen_mappings)))) -drop_freq = dict() -for p in all_droppable: - if drop_freq.get(p): - drop_freq[p] += 1 - else: - drop_freq[p] = 1 +drop_freq = Counter(all_droppable) -for k in sorted(drop_freq.keys(), key=drop_freq.__getitem__, reverse=True)[:20]: - sys.stderr.write("%s: %s\n"%(k, drop_freq[k])) +for package, freq in drop_freq.most_common(20): + sys.stderr.write("%s: %s\n"%(package, freq))