Skip to content

Commit f8e4617

Browse files
committed
Replacer: Dont pass around match object
1 parent 74d1f3b commit f8e4617

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

src/tox/config/__init__.py

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1829,8 +1829,9 @@ def _replace_match(self, match):
18291829

18301830
return os.pathsep
18311831

1832+
default_value = g["default_value"]
18321833
if sub_value == "posargs":
1833-
return self.reader.getposargs(match.group("default_value"))
1834+
return self.reader.getposargs(default_value)
18341835

18351836
sub_type = g["sub_type"]
18361837
if not sub_type and not sub_value:
@@ -1839,27 +1840,26 @@ def _replace_match(self, match):
18391840
"If you were using `{}` for `os.pathsep`, please use `{:}`.",
18401841
)
18411842

1842-
if not sub_type and not g["default_value"] and sub_value == "/":
1843+
if not sub_type and not default_value and sub_value == "/":
18431844
return os.sep
1845+
18441846
if sub_type == "env":
1845-
return self._replace_env(match)
1847+
return self._replace_env(sub_value, default_value)
18461848
if sub_type == "tty":
18471849
if is_interactive():
18481850
return match.group("substitution_value")
18491851
return match.group("default_value")
18501852
if sub_type == "posargs":
1851-
return self.reader.getposargs(match.group("substitution_value"))
1853+
return self.reader.getposargs(sub_value)
18521854
if sub_type is not None:
18531855
raise tox.exception.ConfigError(
18541856
"No support for the {} substitution type".format(sub_type),
18551857
)
1856-
return self._replace_substitution(match)
1858+
return self._replace_substitution(sub_value)
18571859

1858-
def _replace_env(self, match):
1859-
key = match.group("substitution_value")
1860+
def _replace_env(self, key, default):
18601861
if not key:
18611862
raise tox.exception.ConfigError("env: requires an environment variable name")
1862-
default = match.group("default_value")
18631863
value = self.reader.get_environ_value(key)
18641864
if value is not None:
18651865
return value
@@ -1887,8 +1887,7 @@ def _substitute_from_other_section(self, key):
18871887

18881888
raise tox.exception.ConfigError("substitution key {!r} not found".format(key))
18891889

1890-
def _replace_substitution(self, match):
1891-
sub_key = match.group("substitution_value")
1890+
def _replace_substitution(self, sub_key):
18921891
val = self.reader._subs.get(sub_key, None)
18931892
if val is None:
18941893
val = self._substitute_from_other_section(sub_key)

0 commit comments

Comments
 (0)