dan/oxc
1
0
Fork 0
mirror of https://github.com/danbulant/oxc synced 2026-05-19 12:19:15 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(linter): panic in consistent-function-scoping (#5613)

Browse source 
Closes #5608
This commit is contained in:
DonIsaac 2024-09-09 01:21:49 +00:00
parent d22a9b72e0
commit af6d240559
1 changed files with 12 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
crates/oxc_linter/src/rules/unicorn/consistent_function_scoping.rs
View file

@ -185,21 +185,23 @@ impl Rule for ConsistentFunctionScoping {
}),
)
} else if let Some(function_id) = &function.id {
(function_id.symbol_id.get().unwrap(), function_body, function_id.span())
let Some(symbol_id) = function_id.symbol_id.get() else {
return;
};
(symbol_id, function_body, function_id.span())
} else {
return;
}
}
AstKind::ArrowFunctionExpression(arrow_function) if self.check_arrow_functions => {
if let Some(binding_ident) = get_function_like_declaration(node, ctx) {
(
binding_ident.symbol_id.get().unwrap(),
&arrow_function.body,
binding_ident.span(),
)
} else {
let Some(binding_ident) = get_function_like_declaration(node, ctx) else {
return;
}
};
let Some(symbol_id) = binding_ident.symbol_id.get() else {
return;
};
(symbol_id, &arrow_function.body, binding_ident.span())
}
_ => return,
};
@ -586,6 +588,7 @@ fn test() {
("module.exports = function foo() {};", None),
("module.exports.foo = function foo() {};", None),
("foo.bar.func = function foo() {};", None),
("if(f) function f(){}", None),
];
let fail = vec![