diff options
author | Toon Nolten <toon.nolten@student.kuleuven.be> | 2017-05-28 14:49:19 +0200 |
---|---|---|
committer | Toon Nolten <toon.nolten@student.kuleuven.be> | 2017-05-28 14:49:19 +0200 |
commit | 743922694120cb296e3e82a1da66baa0560b5e33 (patch) | |
tree | 2567264d5672cd4921ffdc89ff33a01aa191eb41 /ranger | |
parent | be19d0d41d6a1e3bc8c69d67cdeee8485b04251e (diff) | |
download | ranger-743922694120cb296e3e82a1da66baa0560b5e33.tar.gz |
Resolve pylint issues
Diffstat (limited to 'ranger')
-rwxr-xr-x | ranger/config/commands.py | 56 |
1 files changed, 28 insertions, 28 deletions
diff --git a/ranger/config/commands.py b/ranger/config/commands.py index 33f10b56..314d31c2 100755 --- a/ranger/config/commands.py +++ b/ranger/config/commands.py @@ -91,7 +91,6 @@ import re import subprocess from ranger.api.commands import Command -from ranger.ext.get_executables import get_executables class alias(Command): @@ -1689,36 +1688,37 @@ class yank(Command): } def execute(self): - clipboard_commands = self.clipboards() + def clipboards(): + from ranger.ext.get_executables import get_executables + clipboard_managers = { + 'xclip': [ + ['xclip'], + ['xclip', '-selection', 'clipboard'], + ], + 'xsel': [ + ['xsel'], + ['xsel', '-b'], + ], + 'pbcopy': [ + ['pbcopy'], + ], + } + ordered_managers = ['pbcopy', 'xclip', 'xsel'] + executables = get_executables() + for manager in ordered_managers: + if manager in executables: + return clipboard_managers[manager] + + return [] + + clipboard_commands = clipboards() selection = self.get_selection_attr(self.modes[self.arg(1)]) - input = "\n".join(selection) + input_argument = "\n".join(selection) for command in clipboard_commands: - p = subprocess.Popen(command, universal_newlines=True, - stdin=subprocess.PIPE) - p.communicate(input=input) - - def clipboards(self): - clipboard_managers = { - 'xclip': [ - ['xclip'], - ['xclip', '-selection', 'clipboard'], - ], - 'xsel': [ - ['xsel'], - ['xsel', '-b'], - ], - 'pbcopy': [ - ['pbcopy'], - ], - } - ordered_managers = ['pbcopy', 'xclip', 'xsel'] - executables = get_executables() - for manager in ordered_managers: - if manager in executables: - return clipboard_managers[manager] - else: - return [] + process = subprocess.Popen(command, universal_newlines=True, + stdin=subprocess.PIPE) + process.communicate(input=input_argument) def get_selection_attr(self, attr): return [getattr(file, attr) for file in |