diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2016-12-20 12:12:07 -0800 |
|---|---|---|
| committer | Dylan Baker <dylan@pnwbakers.com> | 2016-12-21 17:18:39 -0800 |
| commit | 41ef1f14636caf213e7c25dc3eeb3d911418ee72 (patch) | |
| tree | 75d34e6edaecaf3ad66fb544e0cd7a996fd4ad4d /alot/completion.py | |
| parent | bf2005c111289fcd7f55b92e64cfa8c43b9bfdbc (diff) | |
Replace unused arguments with _
This patch replaces a large number (but not all) unused arguments with
the standard ``_`` placeholder. This has the advantage of signaling that
these values are unused.
There are a number of places that I've chosen not to apply this, largely
in methods that have publicly define signatures that they inherit
(usually from urwid).
Diffstat (limited to 'alot/completion.py')
| -rw-r--r-- | alot/completion.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/alot/completion.py b/alot/completion.py index 2e643e37..314fa0c6 100644 --- a/alot/completion.py +++ b/alot/completion.py @@ -108,7 +108,7 @@ class MultipleSelectionCompleter(Completer): def complete(self, original, pos): mypart, start, end, mypos = self.relevant_part(original, pos) res = [] - for c, p in self._completer.complete(mypart, mypos): + for c, _ in self._completer.complete(mypart, mypos): newprefix = original[:start] + c if not original[end:].startswith(self._separator): newprefix += self._separator @@ -135,7 +135,7 @@ class QueryCompleter(Completer): myprefix = mypart[:mypos] m = re.search(r'(tag|is|to|from):(\w*)', myprefix) if m: - cmd, params = m.groups() + cmd, _ = m.groups() cmdlen = len(cmd) + 1 # length of the keyword part incld colon if cmd in ['to', 'from']: localres = self._abookscompleter.complete(mypart[cmdlen:], |