rnhmjoj/qutebrowser
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Rename possible_placeholder to arg

Browse Source
This commit is contained in:
Luca Benci 2017-10-18 20:20:05 +02:00
parent 7907840ead
commit cf04219f79
1 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
qutebrowser/misc/editor.py
View File

@ -183,15 +183,15 @@ class ExternalEditor(QObject):
column = caret_position - text[:caret_position].rfind('\n')
return (line, column)
def _sub_placeholder(self, possible_placeholder, line, column):
def _sub_placeholder(self, arg, line, column):
"""Substitute a single placeholder.
If the `possible_placeholder` input to this function is valid it will
If the `arg` input to this function is a valid placeholder it will
be substituted with the appropriate value, otherwise it will be left
unchanged.
Args:
possible_placeholder: an argument of editor.command.
arg: an argument of editor.command.
line: the previously-calculated line number for the text caret.
column: the previously-calculated column number for the text caret.
@ -207,9 +207,7 @@ class ExternalEditor(QObject):
'{column0}': str(column-1)
}
# The for loop would create a copy anyway, this should be more legible
ph = possible_placeholder
for old, new in replacements.items():
ph = ph.replace(old, new)
arg = arg.replace(old, new)
return ph
return arg