Skip to content

Commit d6c02b6

Browse files
Reduce indentation
1 parent b082f10 commit d6c02b6

File tree

1 file changed

+18
-21
lines changed

1 file changed

+18
-21
lines changed

pylint/checkers/utils.py

Lines changed: 18 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1411,28 +1411,25 @@ def function_arguments_are_ambiguous(
14111411
) -> bool:
14121412
if func1.argnames() != func2.argnames():
14131413
return True
1414-
if func1.args.args is not None and func2.args.args is not None:
1415-
# Check ambiguity among function default values
1416-
pairs_of_defaults = [
1417-
(func1.args.defaults, func2.args.defaults),
1418-
(func1.args.kw_defaults, func2.args.kw_defaults),
1419-
]
1420-
for zippable_default in pairs_of_defaults:
1421-
if len(zippable_default[0]) != len(zippable_default[1]):
1422-
return True
1423-
for default1, default2 in zip(*zippable_default):
1424-
if isinstance(default1, nodes.Const) and isinstance(
1425-
default2, nodes.Const
1426-
):
1427-
if default1.value != default2.value:
1428-
return True
1429-
elif isinstance(default1, nodes.Name) and isinstance(
1430-
default2, nodes.Name
1431-
):
1432-
if default1.name != default2.name:
1433-
return True
1434-
else:
1414+
# Check ambiguity among function default values
1415+
pairs_of_defaults = [
1416+
(func1.args.defaults, func2.args.defaults),
1417+
(func1.args.kw_defaults, func2.args.kw_defaults),
1418+
]
1419+
for zippable_default in pairs_of_defaults:
1420+
if None in zippable_default:
1421+
continue
1422+
if len(zippable_default[0]) != len(zippable_default[1]):
1423+
return True
1424+
for default1, default2 in zip(*zippable_default):
1425+
if isinstance(default1, nodes.Const) and isinstance(default2, nodes.Const):
1426+
if default1.value != default2.value:
1427+
return True
1428+
elif isinstance(default1, nodes.Name) and isinstance(default2, nodes.Name):
1429+
if default1.name != default2.name:
14351430
return True
1431+
else:
1432+
return True
14361433
return False
14371434

14381435

0 commit comments

Comments
 (0)