diff options
author | mathieui <mathieui@mathieui.net> | 2016-07-12 21:05:46 +0200 |
---|---|---|
committer | mathieui <mathieui@mathieui.net> | 2016-07-12 21:05:46 +0200 |
commit | a548e2ff6df2abec1791464055deeb9976ca73f8 (patch) | |
tree | 07ca709d8e2dae1b453327b66b38ffa4d98dc913 /poezio/windows | |
parent | d19f53c3e9c5c02773b9fb3cdfe68a3cb245c833 (diff) | |
download | poezio-a548e2ff6df2abec1791464055deeb9976ca73f8.tar.gz poezio-a548e2ff6df2abec1791464055deeb9976ca73f8.tar.bz2 poezio-a548e2ff6df2abec1791464055deeb9976ca73f8.tar.xz poezio-a548e2ff6df2abec1791464055deeb9976ca73f8.zip |
Improve refresh performance in big rooms
going through a list with more than 1000 elements, one at a time, can be
a bit tedious. Especially if you go through it every time you receive
one of those elements.
Diffstat (limited to 'poezio/windows')
-rw-r--r-- | poezio/windows/info_wins.py | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/poezio/windows/info_wins.py b/poezio/windows/info_wins.py index 1d1b5683..4ca72449 100644 --- a/poezio/windows/info_wins.py +++ b/poezio/windows/info_wins.py @@ -213,14 +213,14 @@ class MucInfoWin(InfoWin): def __init__(self): InfoWin.__init__(self) - def refresh(self, room, window=None): + def refresh(self, room, window=None, user=None): log.debug('Refresh: %s', self.__class__.__name__) self._win.erase() self.write_room_name(room) self.write_participants_number(room) self.write_own_nick(room) self.write_disconnected(room) - self.write_role(room) + self.write_role(room, user) if window: self.print_scroll_position(window) self.finish_line(get_theme().COLOR_INFORMATION_BAR) @@ -252,21 +252,16 @@ class MucInfoWin(InfoWin): return self.addstr(truncate_nick(nick, 13), to_curses_attr(get_theme().COLOR_INFORMATION_BAR)) - def write_role(self, room): + def write_role(self, room, user): """ Write our own role and affiliation """ - own_user = None - for user in room.users: - if user.nick == room.own_nick: - own_user = user - break - if not own_user: + if not user: return txt = ' (' - if own_user.affiliation != 'none': - txt += own_user.affiliation+', ' - txt += own_user.role+')' + if user.affiliation != 'none': + txt += user.affiliation + ', ' + txt += user.role + ')' self.addstr(txt, to_curses_attr(get_theme().COLOR_INFORMATION_BAR)) class ConversationStatusMessageWin(InfoWin): |