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

refactor(transformer/react-refresh): small refactor (#6973)

Browse source 
Follow-on after #6937. Refactor 2 small blocks of code.

* Replace 2 `if` statements with 1.
* Obey the mighty clippy!
This commit is contained in:
overlookmotel 2024-10-28 02:57:55 +00:00
parent 610621cadf
commit 9e66c2949d
1 changed files with 19 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
crates/oxc_transformer/src/jsx/refresh.rs
View file

@ -211,14 +211,17 @@ impl<'a, 'ctx> Traverse<'a> for ReactRefresh<'a, 'ctx> {
if !matches!(expr, Expression::CallExpression(_)) {
// Try to get binding from parent VariableDeclarator
let id_binding = if let Ancestor::VariableDeclaratorInit(declarator) = ctx.parent() {
declarator.id().get_binding_identifier().map(BoundIdentifier::from_binding_ident)
} else {
None
};
if let Some(id_binding) = id_binding {
self.handle_function_in_variable_declarator(&id_binding, &binding, arguments, ctx);
return;
if let Ancestor::VariableDeclaratorInit(declarator) = ctx.parent() {
if let Some(ident) = declarator.id().get_binding_identifier() {
let id_binding = BoundIdentifier::from_binding_ident(ident);
self.handle_function_in_variable_declarator(
&id_binding,
&binding,
arguments,
ctx,
);
return;
}
}
}
@ -813,19 +816,17 @@ impl<'a, 'ctx> ReactRefresh<'a, 'ctx> {
);
// Get the address of the statement containing this `VariableDeclarator`
#[allow(clippy::single_match_else)]
let address = match ctx.ancestor(2) {
// For `export const Foo = () => {}`
// which is a `VariableDeclaration` inside a `Statement::ExportNamedDeclaration`
Ancestor::ExportNamedDeclarationDeclaration(export_decl) => export_decl.address(),
// Otherwise just a `const Foo = () => {}`
// which is a `Statement::VariableDeclaration`
_ => {
let address =
if let Ancestor::ExportNamedDeclarationDeclaration(export_decl) = ctx.ancestor(2) {
// For `export const Foo = () => {}`
// which is a `VariableDeclaration` inside a `Statement::ExportNamedDeclaration`
export_decl.address()
} else {
// Otherwise just a `const Foo = () => {}` which is a `Statement::VariableDeclaration`
let var_decl = ctx.ancestor(1);
debug_assert!(matches!(var_decl, Ancestor::VariableDeclarationDeclarations(_)));
var_decl.address()
}
};
};
self.ctx.statement_injector.insert_after(&address, statement);
}