Skip to content

Commit 6a85b99

Browse files
committed
Skip validate_arguments when empty
1 parent 531e41a commit 6a85b99

File tree

2 files changed

+21
-25
lines changed

2 files changed

+21
-25
lines changed

parser/src/python.lalrpop

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -956,18 +956,16 @@ FuncDef: ast::Stmt = {
956956

957957
Parameters: ast::Arguments = {
958958
<location:@L> "(" <a: (ParameterList<TypedParameter, StarTypedParameter>)?> ")" <end_location:@R> =>? {
959-
let args = validate_arguments(
960-
a.unwrap_or_else(|| ast::Arguments {
961-
posonlyargs: vec![],
962-
args: vec![],
963-
vararg: None,
964-
kwonlyargs: vec![],
965-
kw_defaults: vec![],
966-
kwarg: None,
967-
defaults: vec![],
968-
range: optional_range(location, end_location)
969-
})
970-
)?;
959+
let args = a.map(validate_arguments).transpose()?.unwrap_or_else(|| ast::Arguments {
960+
posonlyargs: vec![],
961+
args: vec![],
962+
vararg: None,
963+
kwonlyargs: vec![],
964+
kw_defaults: vec![],
965+
kwarg: None,
966+
defaults: vec![],
967+
range: optional_range(location, end_location)
968+
});
971969

972970
Ok(args)
973971
}

parser/src/python.rs

Lines changed: 11 additions & 13 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)