diff options
author | toonn <toonn@toonn.io> | 2020-01-15 22:15:28 +0100 |
---|---|---|
committer | toonn <toonn@toonn.io> | 2020-07-05 15:23:43 +0200 |
commit | 1cd29c998e735a43ad496e0f26cf9dede1ff22dd (patch) | |
tree | 8b6ddd9896443b9d78b9c428624cdf2f5b19b4bd | |
parent | 401c59dcdcc32986160e908332d0750445f4da87 (diff) | |
download | ranger-1cd29c998e735a43ad496e0f26cf9dede1ff22dd.tar.gz |
Drop unnecessary comprehensions
It's unclear why these comprehensions were used, maybe as a way of cloning the lists? However, this does not seem necessary.
-rwxr-xr-x | ranger/config/commands.py | 7 | ||||
-rw-r--r-- | ranger/core/actions.py | 3 |
2 files changed, 3 insertions, 7 deletions
diff --git a/ranger/config/commands.py b/ranger/config/commands.py index 2d722d0e..f7695d58 100755 --- a/ranger/config/commands.py +++ b/ranger/config/commands.py @@ -337,11 +337,8 @@ class open_with(Command): def execute(self): app, flags, mode = self._get_app_flags_mode(self.rest(1)) - self.fm.execute_file( - files=[f for f in self.fm.thistab.get_selection()], - app=app, - flags=flags, - mode=mode) + self.fm.execute_file(files=self.fm.thistab.get_selection(), + app=app, flags=flags, mode=mode) def tab(self, tabnum): return self._tab_through_executables() diff --git a/ranger/core/actions.py b/ranger/core/actions.py index db4a17c3..6bf8db75 100644 --- a/ranger/core/actions.py +++ b/ranger/core/actions.py @@ -931,8 +931,7 @@ class Actions( # pylint: disable=too-many-instance-attributes,too-many-public-m except IndexError: self.ui.browser.draw_info = [] return - programs = [program for program in self.rifle.list_commands([target.path], None, - skip_ask=True)] + programs = self.rifle.list_commands([target.path], None, skip_ask=True) if programs: num_digits = max((len(str(program[0])) for program in programs)) program_info = ['%s | %s' % (str(program[0]).rjust(num_digits), program[1]) |