diff --git a/gajim/chat_control.py b/gajim/chat_control.py index 441658135..b033ea0df 100644 --- a/gajim/chat_control.py +++ b/gajim/chat_control.py @@ -1428,7 +1428,7 @@ class ChatControl(ChatControlBase): typ = 'pm' self.redraw_after_event_removed(jid) - if (self.contact.show in ('offline', 'error')): + if self.contact.show in ('offline', 'error'): show_offline = app.config.get('showoffline') show_transports = app.config.get('show_transports_group') if (not show_transports and app.jid_is_transport(jid)) or \ diff --git a/gajim/common/helpers.py b/gajim/common/helpers.py index 042a08ff1..7c60fce6d 100644 --- a/gajim/common/helpers.py +++ b/gajim/common/helpers.py @@ -1075,7 +1075,7 @@ def get_notification_icon_tooltip_text(): for account in accounts: account_name = account['name'] # Set account status, if not set above - if (show_more_accounts): + if show_more_accounts: message = '\n' + indent_with + ' %s - %s' text += message % (account_name, get_account_status(account)) # Account list shown, messages need to be indented more diff --git a/gajim/common/resolver.py b/gajim/common/resolver.py index f1cb93f56..95e95d892 100644 --- a/gajim/common/resolver.py +++ b/gajim/common/resolver.py @@ -67,7 +67,7 @@ class CommonResolver(): if host + type_ in self.handlers: for callback in self.handlers[host + type_]: callback(host, result_list) - del(self.handlers[host + type_]) + del self.handlers[host + type_] def start_resolve(self, host, type_): pass diff --git a/gajim/common/socks5.py b/gajim/common/socks5.py index 0063e49b1..93ddc7108 100644 --- a/gajim/common/socks5.py +++ b/gajim/common/socks5.py @@ -390,7 +390,7 @@ class SocksQueue: else: if reader.streamhost is not None: reader.streamhost['state'] = -1 - del(self.readers[key]) + del self.readers[key] def remove_sender_by_key(self, key, do_disconnect=True): sender = self.senders[key] @@ -398,7 +398,7 @@ class SocksQueue: sender.disconnect() self.idlequeue.unplug_idle(sender.fd) self.idlequeue.remove_timeout(sender.fd) - del(self.senders[key]) + del self.senders[key] if self.connected > 0: self.connected -= 1 diff --git a/gajim/common/zeroconf/client_zeroconf.py b/gajim/common/zeroconf/client_zeroconf.py index bc73b4821..6adad3c62 100644 --- a/gajim/common/zeroconf/client_zeroconf.py +++ b/gajim/common/zeroconf/client_zeroconf.py @@ -105,7 +105,7 @@ class ZeroconfListener(IdleObject): for jid in self.conn_holder.getRoster().keys(): entry = self.conn_holder.getRoster().getItem(jid) for address in entry['addresses']: - if (address['address'] == ipaddr): + if address['address'] == ipaddr: from_jid = jid break P2PClient(sock[0], [{'host': ipaddr, 'address': ipaddr, 'port': sock[1][1]}], self.conn_holder, [], from_jid) diff --git a/gajim/common/zeroconf/zeroconf_bonjour.py b/gajim/common/zeroconf/zeroconf_bonjour.py index ff39d5729..3057fee45 100644 --- a/gajim/common/zeroconf/zeroconf_bonjour.py +++ b/gajim/common/zeroconf/zeroconf_bonjour.py @@ -66,7 +66,7 @@ class Zeroconf: if errorCode != pybonjour.kDNSServiceErr_NoError: log.debug('Error in browse_callback: %s', str(errorCode)) return - if not (flags & pybonjour.kDNSServiceFlagsAdd): + if not flags & pybonjour.kDNSServiceFlagsAdd: self.remove_service_callback(serviceName) return diff --git a/gajim/config.py b/gajim/config.py index c72f1e166..2e8e269bc 100644 --- a/gajim/config.py +++ b/gajim/config.py @@ -109,7 +109,7 @@ class ManageProxiesWindow: model = self.proxies_treeview.get_model() proxies = app.config.get_per('proxies') i = 1 - while ('proxy' + str(i)) in proxies: + while 'proxy' + str(i) in proxies: i += 1 iter_ = model.append() model.set(iter_, 0, 'proxy' + str(i)) diff --git a/gajim/dialogs.py b/gajim/dialogs.py index 3ffc969db..e98cfe14b 100644 --- a/gajim/dialogs.py +++ b/gajim/dialogs.py @@ -751,8 +751,8 @@ class ChangeStatusMessageDialog(TimeoutDialog): def on_change_status_message_dialog_key_press_event(self, widget, event): self.countdown_enabled = False if event.keyval == Gdk.KEY_Return or \ - event.keyval == Gdk.KEY_KP_Enter: # catch CTRL+ENTER - if (event.get_state() & Gdk.ModifierType.CONTROL_MASK): + event.keyval == Gdk.KEY_KP_Enter: # catch CTRL+ENTER + if event.get_state() & Gdk.ModifierType.CONTROL_MASK: self.dialog.response(Gtk.ResponseType.OK) # Stop the event return True diff --git a/gajim/filetransfers_window.py b/gajim/filetransfers_window.py index 739fe2e55..33e2480d8 100644 --- a/gajim/filetransfers_window.py +++ b/gajim/filetransfers_window.py @@ -577,7 +577,7 @@ class FileTransfersWindow: app.interface.roster.draw_contact(jid, account) app.interface.roster.show_title() FilesProp.deleteFileProp(file_props) - del(file_props) + del file_props def set_progress(self, typ, sid, transfered_size, iter_=None): """ diff --git a/gajim/gajim_remote.py b/gajim/gajim_remote.py index c2ae86c5f..c9c38d62b 100644 --- a/gajim/gajim_remote.py +++ b/gajim/gajim_remote.py @@ -436,7 +436,7 @@ class GajimRemote: res = self.print_info(level+1, val) if res != '': ret_str += '%s%s: \n%s' % (spacing, key, res) - if (encode_return): + if encode_return: try: ret_str = ret_str.encode(PREFERRED_ENCODING) except Exception: diff --git a/gajim/groupchat_control.py b/gajim/groupchat_control.py index 3a9e9386b..41904ec24 100644 --- a/gajim/groupchat_control.py +++ b/gajim/groupchat_control.py @@ -1474,7 +1474,7 @@ class GroupchatControl(ChatControlBase): for special_word in special_words: found_here = text.find(special_word) - while(found_here > -1): + while found_here > -1: end_here = found_here + len(special_word) if (found_here == 0 or not text[found_here - 1].isalpha()) and \ (end_here == len(text) or not text[end_here].isalpha()): @@ -2783,7 +2783,7 @@ class GroupchatControl(ChatControlBase): is set this way) """ if path.get_depth() == 1: # It's a group - if (widget.row_expanded(path)): + if widget.row_expanded(path): widget.collapse_row(path) else: widget.expand_row(path, False) @@ -2834,7 +2834,7 @@ class GroupchatControl(ChatControlBase): self.room_jid): # it's a group if x < 27: - if (widget.row_expanded(path)): + if widget.row_expanded(path): widget.collapse_row(path) else: widget.expand_row(path, False) diff --git a/gajim/gtk/emoji_data.py b/gajim/gtk/emoji_data.py index 9dcf6ca43..a88d70d7a 100644 --- a/gajim/gtk/emoji_data.py +++ b/gajim/gtk/emoji_data.py @@ -145,7 +145,7 @@ class EmojiAsciiData(dict): # escape regexp metachars emoticon_escaped = re.escape(emoticon) emoticons_pattern += emoticon_escaped + '|' - if (emoticon_length != len(emoticon)): + if emoticon_length != len(emoticon): # Build up expressions to match emoticons next to others pre_pattern = pre_pattern[:-1] + ')|(?<=' post_pattern = post_pattern[:-1] + ')|(?=' diff --git a/gajim/gtk/history.py b/gajim/gtk/history.py index 3ee9e9346..92d4ece7b 100644 --- a/gajim/gtk/history.py +++ b/gajim/gtk/history.py @@ -412,7 +412,7 @@ class HistoryWindow: log = time.localtime(timestamp) y, m, d = log[0], log[1], log[2] date = datetime.datetime(y, m, d) - return(date) + return date def _change_date(self, widget): # Get day selected in calendar diff --git a/gajim/gtkgui_helpers.py b/gajim/gtkgui_helpers.py index 9e5f57a1e..ccffe3fde 100644 --- a/gajim/gtkgui_helpers.py +++ b/gajim/gtkgui_helpers.py @@ -732,7 +732,7 @@ def make_jabber_state_images(): app.interface.jabber_state_images['32'] = load_iconset(path) path = os.path.join(helpers.get_iconset_path(iconset), '24x24') - if (os.path.exists(path)): + if os.path.exists(path): app.interface.jabber_state_images['24'] = load_iconset(path) else: # Resize 32x32 icons to 24x24 diff --git a/gajim/gui_interface.py b/gajim/gui_interface.py index 35dce97af..5b4517c67 100644 --- a/gajim/gui_interface.py +++ b/gajim/gui_interface.py @@ -358,7 +358,7 @@ class Interface: elif (obj.errcode == '404') or (obj.errcon in ('item-not-found', 'remote-server-not-found')): # remote server does not exist - if (obj.errcon == 'remote-server-not-found'): + if obj.errcon == 'remote-server-not-found': self.handle_gc_error(gc_control, _('Unable to join group chat'), _('Remote server %s does not exist.') % obj.room_jid) # group chat does not exist diff --git a/gajim/htmltextview.py b/gajim/htmltextview.py index ab5530c51..05db049ad 100644 --- a/gajim/htmltextview.py +++ b/gajim/htmltextview.py @@ -1063,7 +1063,7 @@ class HtmlTextView(Gtk.TextView): if bounds: (search_iter, end) = bounds - while (search_iter.compare(end)): + while search_iter.compare(end): character = search_iter.get_char() if character == '\ufffc': anchor = search_iter.get_child_anchor() diff --git a/gajim/message_window.py b/gajim/message_window.py index 7bfd3b72c..05cfc6687 100644 --- a/gajim/message_window.py +++ b/gajim/message_window.py @@ -722,7 +722,7 @@ class MessageWindow: return self._widget_to_control(nth_child) def has_control(self, jid, acct): - return (acct in self._controls and jid in self._controls[acct]) + return acct in self._controls and jid in self._controls[acct] def change_key(self, old_jid, new_jid, acct): """ diff --git a/gajim/roster_window.py b/gajim/roster_window.py index c3737b39d..e715e4300 100644 --- a/gajim/roster_window.py +++ b/gajim/roster_window.py @@ -1149,7 +1149,7 @@ class RosterWindow: # Hide group if no more contacts iterG = self._get_group_iter(group, account, model=self.modelfilter) to_hide = [] - while(iterG): + while iterG: parent = self.modelfilter.iter_parent(iterG) if (not self.modelfilter.iter_has_child(iterG)) or (to_hide \ and self.modelfilter.iter_n_children(iterG) == 1): @@ -3220,8 +3220,9 @@ class RosterWindow: elif type_ == 'agent': self.on_remove_agent(widget, list_) - elif not (event.get_state() & (Gdk.ModifierType.CONTROL_MASK | \ - Gdk.ModifierType.MOD1_MASK)): + elif not (event.get_state() & + (Gdk.ModifierType.CONTROL_MASK | + Gdk.ModifierType.MOD1_MASK)): num = Gdk.keyval_to_unicode(event.keyval) if num and num > 31: # if we got unicode symbol without ctrl / alt @@ -3362,7 +3363,7 @@ class RosterWindow: titer = model.get_iter(path) if x > x_min and x < x_min + 27 and type_ == 'contact' and \ model.iter_has_child(titer): - if (self.tree.row_expanded(path)): + if self.tree.row_expanded(path): self.tree.collapse_row(path) else: self.tree.expand_row(path, False) @@ -3375,13 +3376,13 @@ class RosterWindow: else: if type_ == 'group' and x < 27: # first cell in 1st column (the arrow SINGLE clicked) - if (self.tree.row_expanded(path)): + if self.tree.row_expanded(path): self.tree.collapse_row(path) else: self.expand_group_row(path) elif type_ == 'contact' and x > x_min and x < x_min + 27: - if (self.tree.row_expanded(path)): + if self.tree.row_expanded(path): self.tree.collapse_row(path) else: self.tree.expand_row(path, False)