8000 gh-102378: don't bother stripping `/` from __text_signature__ · python/cpython@5403c68 · GitHub
[go: up one dir, main page]
Skip to content

Commit 5403c68

Browse files
davidhewittdavidhewitt
committed
gh-102378: don't bother stripping / from __text_signature__
1 parent 71db5db commit 5403c68

File tree

2 files changed

+23
-43
lines changed

2 files changed

+23
-43
lines changed

‎Lib/inspect.py

Lines changed: 16 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -2106,19 +2106,16 @@ def _signature_strip_non_python_syntax(signature):
21062106
Private helper function. Takes a signature in Argument Clinic's
21072107
extended signature format.
21082108
2109-
Returns a tuple of three things:
2110-
* that signature re-rendered in standard Python syntax,
2109+
Returns a tuple of two things:
2110+
* that signature re-rendered in standard Python syntax, and
21112111
* the index of the "self" parameter (generally 0), or None if
2112-
the function does not have a "self" parameter, and
2113-
* the index of the last "positional only" parameter,
2114-
or None if the signature has no positional-only parameters.
2112+
the function does not have a "self" parameter.
21152113
"""
21162114

21172115
if not signature:
2118-
return signature, None, None
2116+
return signature, None
21192117

21202118
self_parameter = None
2121-
last_positional_only = None
21222119

21232120
lines = [l.encode('ascii') for l in signature.split('\n') if l]
21242121
generator = iter(lines).__next__
@@ -2150,13 +2147,6 @@ def _signature_strip_non_python_syntax(signature):
21502147
current_parameter += 1
21512148
continue
21522149

2153-
if string == '/':
2154-
assert not skip_next_comma
2155-
assert last_positional_only is None
2156-
skip_next_comma = True
2157-
last_positional_only = current_parameter - 1
2158-
continue
2159-
21602150
if (type == ERRORTOKEN) and (string == '$'):
21612151
assert self_parameter is None
21622152
self_parameter = current_parameter
@@ -2170,7 +2160,7 @@ def _signature_strip_non_python_syntax(signature):
21702160
if (string == ','):
21712161
add(' ')
21722162
clean_signature = ''.join(text)
2173-
return clean_signature, self_parameter, last_positional_only
2163+
return clean_signature, self_parameter
21742164

21752165

21762166
def _signature_fromstr(cls, obj, s, skip_bound_arg=True):
@@ -2179,8 +2169,7 @@ def _signature_fromstr(cls, obj, s, skip_bound_arg=True):
21792169
"""
21802170
Parameter = cls._parameter_cls
21812171

2182-
clean_signature, self_parameter, last_positional_only = \
2183-
_signature_strip_non_python_syntax(s)
2172+
clean_signature, self_parameter = _signature_strip_non_python_syntax(s)
21842173

21852174
program = "def foo" + clean_signature + ": pass"
21862175

@@ -2269,17 +2258,17 @@ def p(name_node, default_node, default=empty):
22692258
parameters.append(Parameter(name, kind, default=default, annotation=empty))
22702259

22712260
# non-keyword-only parameters
2272-
args = reversed(f.args.args)
2273-
defaults = reversed(f.args.defaults)
2274-
iter = itertools.zip_longest(args, defaults, fillvalue=None)
2275-
if last_positional_only is not None:
2276-
kind = Parameter.POSITIONAL_ONLY
2277-
else:
2278-
kind = Parameter.POSITIONAL_OR_KEYWORD
2279-
for i, (name, default) in enumerate(reversed(list(iter))):
2261+
total_non_kw_args = len(f.args.posonlyargs) + len(f.args.args)
2262+
required_non_kw_args = total_non_kw_args - len(f.args.defaults)
2263+
defaults = itertools.chain(itertools.repeat(None, required_non_kw_args), f.args.defaults)
2264+
2265+
kind = Parameter.POSITIONAL_ONLY
2266+
for (name, default) in zip(f.args.posonlyargs, defaults):
2267+
p(name, default)
2268+
2269+
kind = Parameter.POSITIONAL_OR_KEYWORD
2270+
for (name, default) in zip(f.args.args, defaults):
22802271
p(name, default)
2281-
if i == last_positional_only:
2282-
kind = Parameter.POSITIONAL_OR_KEYWORD
22832272

22842273
# *args
22852274
if f.args.vararg:

‎Lib/test/test_inspect.py

Lines changed: 7 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -4230,56 +4230,47 @@ def foo(a): pass
42304230

42314231
class TestSignaturePrivateHelpers(unittest.TestCase):
42324232
def _strip_non_python_syntax(self, input,
4233-
clean_signature, self_parameter, last_positional_only):
4233+
clean_signature, self_parameter):
42344234
computed_clean_signature, \
4235-
computed_self_parameter, \
4236-
computed_last_positional_only = \
4235+
computed_self_parameter = \
42374236
inspect._signature_strip_non_python_syntax(input)
42384237
self.assertEqual(computed_clean_signature, clean_signature)
42394238
self.assertEqual(computed_self_parameter, self_parameter)
4240-
self.assertEqual(computed_last_positional_only, last_positional_only)
42414239

42424240
def test_signature_strip_non_python_syntax(self):
42434241
self._strip_non_python_syntax(
42444242
"($module, /, path, mode, *, dir_fd=None, " +
42454243
"effective_ids=False,\n follow_symlinks=True)",
4246-
"(module, path, mode, *, dir_fd=None, " +
4244+
"(module, /, path, mode, *, dir_fd=None, " +
42474245
"effective_ids=False, follow_symlinks=True)",
4248-
0,
42494246
0)
42504247

42514248
self._strip_non_python_syntax(
42524249
"($module, word, salt, /)",
4253-
"(module, word, salt)",
4254-
0,
4255-
2)
4250+
"(module, word, salt, /)",
4251+
0)
42564252

42574253
self._strip_non_python_syntax(
42584254
"(x, y=None, z=None, /)",
4259-
"(x, y=None, z=None)",
4260-
None,
4261-
2)
4255+
"(x, y=None, z=None, /)",
4256+
None)
42624257

42634258
self._strip_non_python_syntax(
42644259
"(x, y=None, z=None)",
42654260
"(x, y=None, z=None)",
4266-
None,
42674261
None)
42684262

42694263
self._strip_non_python_syntax(
42704264
"(x,\n y=None,\n z = None )",
42714265
"(x, y=None, z=None)",
4272-
None,
42734266
None)
42744267

42754268
self._strip_non_python_syntax(
42764269
"",
42774270
"",
4278-
None,
42794271
None)
42804272

42814273
self._strip_non_python_syntax(
4282-
None,
42834274
None,
42844275
None,
42854276
None)

0 commit comments

Comments
 (0)
0