Use tuples instead of lists

sebastinas · sebastinas · commit be8737c1dd50 · 2020-10-21T01:00:33.000+02:00
diff --git a/bpython/cli.py b/bpython/cli.py
@@ -190,7 +190,7 @@ def readline(self, size=-1):
         try:
             while not buffer.endswith(("\n", "\r")):
                 key = self.interface.get_key()
-                if key in [curses.erasechar(), "KEY_BACKSPACE"]:
+                if key in (curses.erasechar(), "KEY_BACKSPACE"):
                     y, x = self.interface.scr.getyx()
                     if buffer:
                         self.interface.scr.delch(y, x - 1)
diff --git a/bpython/curtsiesfrontend/interaction.py b/bpython/curtsiesfrontend/interaction.py
@@ -87,7 +87,7 @@ def process_event(self, e):
             for ee in e.events:
                 # strip control seq
                 self.add_normal_character(ee if len(ee) == 1 else ee[-1])
-        elif e in ["<ESC>"] or isinstance(e, events.SigIntEvent):
+        elif e == "<ESC>" or isinstance(e, events.SigIntEvent):
             self.request_context.switch(False)
             self.escape()
         elif e in edit_keys:
diff --git a/bpython/curtsiesfrontend/repl.py b/bpython/curtsiesfrontend/repl.py
@@ -136,10 +136,9 @@ def process_event(self, e):
             self.repl.run_code_and_maybe_finish()
         elif e in ("<Esc+.>",):
             self.get_last_word()
-
-        elif e in ["<ESC>"]:
+        elif e in ("<ESC>",):
             pass
-        elif e in ["<Ctrl-d>"]:
+        elif e in ("<Ctrl-d>",):
             if self.current_line == "":
                 self.repl.send_to_stdin("\n")
                 self.has_focus = False
@@ -148,7 +147,7 @@ def process_event(self, e):
                 self.repl.run_code_and_maybe_finish(for_code="")
             else:
                 pass
-        elif e in ["\n", "\r", "<Ctrl-j>", "<Ctrl-m>"]:
+        elif e in ("\n", "\r", "<Ctrl-j>", "<Ctrl-m>"):
             line = self.current_line
             self.repl.send_to_stdin(line + "\n")
             self.has_focus = False
@@ -164,7 +163,7 @@ def process_event(self, e):
             self.repl.send_to_stdin(self.current_line)
 
     def add_input_character(self, e):
-        if e == "<SPACE>":
+        if e in ("<SPACE>",):
             e = " "
         if e.startswith("<") and e.endswith(">"):
             return
@@ -763,7 +762,7 @@ def process_key_event(self, e):
             raise SystemExit()
         elif e in ("\n", "\r", "<PADENTER>", "<Ctrl-j>", "<Ctrl-m>"):
             self.on_enter()
-        elif e == "<TAB>":  # tab
+        elif e in ("<TAB>",):  # tab
             self.on_tab()
         elif e in ("<Shift-TAB>",):
             self.on_tab(back=True)
@@ -784,9 +783,9 @@ def process_key_event(self, e):
         # TODO add PAD keys hack as in bpython.cli
         elif e in key_dispatch[self.config.edit_current_block_key]:
             self.send_current_block_to_external_editor()
-        elif e in ["<ESC>"]:
+        elif e in ("<ESC>",):
             self.incr_search_mode = None
-        elif e in ["<SPACE>"]:
+        elif e in ("<SPACE>",):
             self.add_normal_character(" ")
         else:
             self.add_normal_character(e)
@@ -969,7 +968,7 @@ def process_simple_keypress(self, e):
                 self.process_event(bpythonevents.RefreshRequestEvent())
         elif isinstance(e, events.Event):
             pass  # ignore events
-        elif e == "<SPACE>":
+        elif e in ("<SPACE>",):
             self.add_normal_character(" ")
         else:
             self.add_normal_character(e)
@@ -2013,11 +2012,11 @@ def key_help_text(self):
             ["complete history suggestion", "right arrow at end of line"]
         )
         pairs.append(["previous match with current line", "up arrow"])
-        for functionality, key in [
+        for functionality, key in (
             (attr[:-4].replace("_", " "), getattr(self.config, attr))
             for attr in self.config.__dict__
             if attr.endswith("key")
-        ]:
+        ):
             if functionality in NOT_IMPLEMENTED:
                 key = "Not Implemented"
             if key == "":
@@ -2070,7 +2069,7 @@ def just_simple_events(event_list):
             simple_events.append("\n")
         elif isinstance(e, events.Event):
             pass  # ignore events
-        elif e == "<SPACE>":
+        elif e in ("<SPACE>",):
             simple_events.append(" ")
         elif len(e) > 1:
             pass  # get rid of <Ctrl-a> etc.
diff --git a/bpython/lazyre.py b/bpython/lazyre.py
@@ -20,7 +20,6 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 # THE SOFTWARE.
 
-
 import re
 
 
diff --git a/bpython/repl.py b/bpython/repl.py
@@ -21,7 +21,6 @@
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 # THE SOFTWARE.
 
-
 import code
 import inspect
 import os
@@ -40,9 +39,7 @@
 
 from pygments.token import Token
 from pygments.lexers import Python3Lexer
-
-from . import autocomplete
-from . import inspection
+from . import autocomplete, inspection, simpleeval
 from .clipboard import get_clipboard, CopyFailed
 from .config import getpreferredencoding
 from .formatter import Parenthesis
@@ -51,7 +48,6 @@
 from .paste import PasteHelper, PastePinnwand, PasteFailed
 from .patch_linecache import filename_for_console_input
 from .translations import _, ngettext
-from . import simpleeval
 
 
 class RuntimeTimer:
@@ -1206,7 +1202,7 @@ def next_token_inside_string(code_string, inside_string):
     for token, value in Python3Lexer().get_tokens(code_string):
         if token is Token.String:
             value = value.lstrip("bBrRuU")
-            if value in ['"""', "'''", '"', "'"]:
+            if value in ('"""', "'''", '"', "'"):
                 if not inside_string:
                     inside_string = value
                 elif value == inside_string:
diff --git a/bpython/urwid.py b/bpython/urwid.py
@@ -393,7 +393,7 @@ def move_cursor_to_coords(self, *args):
         return False
 
     def keypress(self, size, key):
-        if urwid.command_map[key] in ["cursor up", "cursor down"]:
+        if urwid.command_map[key] in ("cursor up", "cursor down"):
             # Do not handle up/down arrow, leave them for the repl.
             return key
 
@@ -452,7 +452,7 @@ class BPythonListBox(urwid.ListBox):
     """
 
     def keypress(self, size, key):
-        if key not in ["up", "down"]:
+        if key not in ("up", "down"):
             return urwid.ListBox.keypress(self, size, key)
         return key
 
