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

fix(transformer/nullish-coalescing): correct span (#7269)

Browse source 
Transformed expression inherit same `Span` as the input.
This commit is contained in:
overlookmotel 2024-11-14 16:17:22 +00:00
parent e219ae8c94
commit 5b5c8a9bdb
1 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
crates/oxc_transformer/src/es2020/nullish_coalescing_operator.rs
View file

@ -67,6 +67,7 @@ impl<'a, 'ctx> Traverse<'a> for NullishCoalescingOperator<'a, 'ctx> {
Self::clone_expression(&logical_expr.left, ctx),
logical_expr.left,
logical_expr.right,
logical_expr.span,
ctx,
);
return;
@ -98,8 +99,13 @@ impl<'a, 'ctx> Traverse<'a> for NullishCoalescingOperator<'a, 'ctx> {
binding.create_read_write_target(ctx),
logical_expr.left,
);
let mut new_expr =
Self::create_conditional_expression(reference, assignment, logical_expr.right, ctx);
let mut new_expr = Self::create_conditional_expression(
reference,
assignment,
logical_expr.right,
logical_expr.span,
ctx,
);
if is_parent_formal_parameter {
// Replace `function (a, x = a.b ?? c) {}` to `function (a, x = (() => a.b ?? c)() ){}`
@ -173,6 +179,7 @@ impl<'a, 'ctx> NullishCoalescingOperator<'a, 'ctx> {
reference: Expression<'a>,
assignment: Expression<'a>,
default: Expression<'a>,
span: Span,
ctx: &mut TraverseCtx<'a>,
) -> Expression<'a> {
let op = BinaryOperator::StrictInequality;
@ -186,6 +193,6 @@ impl<'a, 'ctx> NullishCoalescingOperator<'a, 'ctx> {
);
let test = ctx.ast.expression_logical(SPAN, left, LogicalOperator::And, right);
ctx.ast.expression_conditional(SPAN, test, reference, default)
ctx.ast.expression_conditional(span, test, reference, default)
}
}