little simplifying

2 files changed, 6 insertions, 10 deletions

diff --git a/lint.py b/lint.py
index abe7411..97b0e24 100644
--- a/lint.py
+++ b/lint.py
@@ -456,8 +456,6 @@ This is used by the global evaluation report (R0004).'}),
             files_or_modules = (files_or_modules,)
         filemods = self.expand_files(files_or_modules)
         checkers = sort_checkers(self._checkers.values())
-        rev_checkers = checkers[:]
-        rev_checkers.reverse()
         # notify global begin
         for checker in checkers:
             checker.open()
@@ -492,7 +490,8 @@ This is used by the global evaluation report (R0004).'}),
             self.check_astng_module(astng, checkers)
         # notify global end
         self.set_current_module('')
-        for checker in rev_checkers:
+        checkers.reverse()
+        for checker in  checkers:
             checker.close()
 
     def expand_files(self, files_or_modules):
diff --git a/utils.py b/utils.py
index 022807c..5123187 100644
--- a/utils.py
+++ b/utils.py
@@ -44,15 +44,12 @@ MSG_TYPES = {
 MSG_CATEGORIES = MSG_TYPES.keys()
 
 
-def sort_checkers(checkers, enabled_only=True):
+def sort_checkers(checkers, enabled_only=True): # XXX enabled_only never used ?
     """return a list of enabled checker sorted by priority"""
     if enabled_only:
-        checkers = [(-checker.priority, checker) for checker in checkers
-                    if checker.is_enabled()]
-    else:
-        checkers = [(-checker.priority, checker) for checker in checkers]
-    checkers.sort()
-    return [item[1] for item in checkers]
+        checkers = [checker for checker in checkers if checker.is_enabled()]
+    checkers.sort(lambda x, y: cmp(-x.priority, -y.priority) )
+    return checkers
 
 def sort_msgs(msg_ids):
     """sort message identifiers according to their category first"""