From 69b4480e9e6ff578b880b7d8089616bc68e60f71 Mon Sep 17 00:00:00 2001
From: Emmanuel Gil Peyrot <linkmauve@linkmauve.fr>
Date: Wed, 6 Jul 2016 04:23:31 +0100
Subject: Make UserList.refresh_if_possible not hyper dumb.

This function was actually calling refresh for every single item that
was different from the cached version.
---
 poezio/windows/muc.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/poezio/windows/muc.py b/poezio/windows/muc.py
index 897b9f81..ee57ab6e 100644
--- a/poezio/windows/muc.py
+++ b/poezio/windows/muc.py
@@ -47,10 +47,11 @@ class UserList(Win):
             self.cache = new
             self.refresh(users)
             return
-        for i in range(len(old)):
-            if old[i] != new[i]:
+        for a, b in zip(old, new):
+            if a != b:
                 self.cache = new
                 self.refresh(users)
+                return
 
     def refresh(self, users):
         log.debug('Refresh: %s', self.__class__.__name__)
-- 
cgit v1.2.3