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/class-properties): rename *_owner to owned_* (#7855)

Browse source 
If I understand https://github.com/oxc-project/oxc/pull/7831?notification_referrer_id=NT_kwDOAcKkeLQxMzgzNzE1MTAwNjoyOTUzMzMwNA#pullrequestreview-2502317967 correctly, `owned_*` are better here
This commit is contained in:
Dunqing 2024-12-13 15:28:55 +00:00
parent 4b2433539b
commit 0f5e0787b8
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

@ -1132,8 +1132,8 @@ impl<'a, 'ctx> ClassProperties<'a, 'ctx> {
// `o?.Foo.#self.self?.self.unicorn;` -> `(result ? void 0 : object)?.self.unicorn`
// ^^^^^^^^^^^^^^^^^ the object has transformed, if the current member is optional,
// then we need to wrap it to a conditional expression
let object_owner = ctx.ast.move_expression(object);
*object = Self::wrap_conditional_check(result, object_owner, ctx);
let owned_object = ctx.ast.move_expression(object);
*object = Self::wrap_conditional_check(result, owned_object, ctx);
None
} else {
Some(result)
@ -1359,8 +1359,8 @@ impl<'a, 'ctx> ClassProperties<'a, 'ctx> {
// After the below transformation, the `callee` will be a temp variable.
let result = self.transform_expression_to_wrap_nullish_check(callee, ctx);
let callee_owner = ctx.ast.move_expression(callee);
Self::substitute_callee_and_insert_context(call, callee_owner, context, ctx);
let owned_callee = ctx.ast.move_expression(callee);
Self::substitute_callee_and_insert_context(call, owned_callee, context, ctx);
result
}