Skip to content

Commit a7cf856

Browse files
committed
refactor(ast/estree): shorten raw transfer deserializer for AssignmentTargetPropertyIdentifier (#15303)
There are no `deserialize` calls while constructing `value` here, so it's pointless to assign to `parent` and then set it back again. Remove this pointless code, and shorten the deserializer a little.
1 parent d92451e commit a7cf856

File tree

6 files changed

+11
-23
lines changed

6 files changed

+11
-23
lines changed

apps/oxlint/src-js/generated/deserialize.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1254,9 +1254,8 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
12541254
init = deserializeOptionExpression(pos + 40),
12551255
value = { ...key };
12561256
if (init !== null) {
1257-
let left = value,
1258-
previousParent = parent;
1259-
value = parent = {
1257+
let left = value;
1258+
value = {
12601259
__proto__: NodeProto,
12611260
type: 'AssignmentPattern',
12621261
decorators: [],
@@ -1271,7 +1270,6 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
12711270
};
12721271
left.parent = value;
12731272
init.parent = value;
1274-
parent = previousParent;
12751273
}
12761274
node.kind = 'init';
12771275
node.key = key;

crates/oxc_ast/src/serialize/js.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -430,8 +430,7 @@ impl ESTree for ArrowFunctionExpressionBody<'_> {
430430
let value = { ...THIS.key };
431431
if (init !== null) {
432432
const left = value;
433-
const previousParent = parent;
434-
value = parent = {
433+
value = {
435434
type: 'AssignmentPattern',
436435
...(IS_TS && { decorators: [] }),
437436
left,
@@ -448,7 +447,6 @@ impl ESTree for ArrowFunctionExpressionBody<'_> {
448447
if (PARENT) {
449448
left.parent = value;
450449
init.parent = value;
451-
parent = previousParent;
452450
}
453451
}
454452
value

napi/parser/generated/deserialize/js_parent.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1096,9 +1096,8 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
10961096
init = deserializeOptionExpression(pos + 40),
10971097
value = { ...key };
10981098
if (init !== null) {
1099-
let left = value,
1100-
previousParent = parent;
1101-
value = parent = {
1099+
let left = value;
1100+
value = {
11021101
type: 'AssignmentPattern',
11031102
left,
11041103
right: init,
@@ -1108,7 +1107,6 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
11081107
};
11091108
left.parent = value;
11101109
init.parent = value;
1111-
parent = previousParent;
11121110
}
11131111
node.kind = 'init';
11141112
node.key = key;

napi/parser/generated/deserialize/js_range_parent.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1129,9 +1129,8 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
11291129
init = deserializeOptionExpression(pos + 40),
11301130
value = { ...key };
11311131
if (init !== null) {
1132-
let left = value,
1133-
previousParent = parent;
1134-
value = parent = {
1132+
let left = value;
1133+
value = {
11351134
type: 'AssignmentPattern',
11361135
left,
11371136
right: init,
@@ -1142,7 +1141,6 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
11421141
};
11431142
left.parent = value;
11441143
init.parent = value;
1145-
parent = previousParent;
11461144
}
11471145
node.kind = 'init';
11481146
node.key = key;

napi/parser/generated/deserialize/ts_parent.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1170,9 +1170,8 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
11701170
init = deserializeOptionExpression(pos + 40),
11711171
value = { ...key };
11721172
if (init !== null) {
1173-
let left = value,
1174-
previousParent = parent;
1175-
value = parent = {
1173+
let left = value;
1174+
value = {
11761175
type: 'AssignmentPattern',
11771176
decorators: [],
11781177
left,
@@ -1185,7 +1184,6 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
11851184
};
11861185
left.parent = value;
11871186
init.parent = value;
1188-
parent = previousParent;
11891187
}
11901188
node.kind = 'init';
11911189
node.key = key;

napi/parser/generated/deserialize/ts_range_parent.js

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1203,9 +1203,8 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
12031203
init = deserializeOptionExpression(pos + 40),
12041204
value = { ...key };
12051205
if (init !== null) {
1206-
let left = value,
1207-
previousParent = parent;
1208-
value = parent = {
1206+
let left = value;
1207+
value = {
12091208
type: 'AssignmentPattern',
12101209
decorators: [],
12111210
left,
@@ -1219,7 +1218,6 @@ function deserializeAssignmentTargetPropertyIdentifier(pos) {
12191218
};
12201219
left.parent = value;
12211220
init.parent = value;
1222-
parent = previousParent;
12231221
}
12241222
node.kind = 'init';
12251223
node.key = key;

0 commit comments

Comments
 (0)