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

refactor(transformer/class-properties): remove a branch from transform_call_expression_impl (#7507)

Browse source 
Small optimization. `transform_call_expression_impl` does not require the `Expression::CallExpression`, only the `CallExpression` itself. So pass that into the function, instead of having to unwrap it (which is an unnecessary branch).
This commit is contained in:
overlookmotel 2024-11-27 11:24:10 +00:00
parent 1c4b29c100
commit 5261547c25
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
crates/oxc_transformer/src/es2022/class_properties/private.rs
View file

@ -128,21 +128,21 @@ impl<'a, 'ctx> ClassProperties<'a, 'ctx> {
) {
let Expression::CallExpression(call_expr) = expr else { unreachable!() };
if matches!(&call_expr.callee, Expression::PrivateFieldExpression(_)) {
self.transform_call_expression_impl(expr, ctx);
self.transform_call_expression_impl(call_expr, ctx);
};
}
fn transform_call_expression_impl(
&mut self,
expr: &mut Expression<'a>,
call_expr: &mut CallExpression<'a>,
ctx: &mut TraverseCtx<'a>,
) {
// Unfortunately no way to make compiler see that these branches are provably unreachable.
// Unfortunately no way to make compiler see that this branch is provably unreachable.
// This function is much too large to inline.
let Expression::CallExpression(call_expr) = expr else { unreachable!() };
let Expression::PrivateFieldExpression(field_expr) = &mut call_expr.callee else {
unreachable!()
};
let prop_details = self.lookup_private_property(&field_expr.field);
// TODO: Should never be `None` - only because implementation is incomplete.
let Some((prop, class_name_binding, is_declaration)) = prop_details else { return };