From 4ce018b0b5f7b861945413dd6431d4c55fde8854 Mon Sep 17 00:00:00 2001 From: Nguyen Tran Date: Sat, 11 Mar 2023 16:35:46 -0500 Subject: [PATCH] Remove unused owner field in unpack_destructuring --- src/compiler/compile/nodes/AwaitBlock.ts | 4 ++-- src/compiler/compile/nodes/ConstTag.ts | 3 +-- src/compiler/compile/nodes/EachBlock.ts | 2 +- src/compiler/compile/nodes/shared/Context.ts | 9 --------- 4 files changed, 4 insertions(+), 14 deletions(-) diff --git a/src/compiler/compile/nodes/AwaitBlock.ts b/src/compiler/compile/nodes/AwaitBlock.ts index 56471cde2c..4a669b6365 100644 --- a/src/compiler/compile/nodes/AwaitBlock.ts +++ b/src/compiler/compile/nodes/AwaitBlock.ts @@ -35,12 +35,12 @@ export default class AwaitBlock extends Node { if (this.then_node) { this.then_contexts = []; - unpack_destructuring({ contexts: this.then_contexts, node: info.value, scope, component, context_rest_properties: this.context_rest_properties, owner: this }); + unpack_destructuring({ contexts: this.then_contexts, node: info.value, scope, component, context_rest_properties: this.context_rest_properties }); } if (this.catch_node) { this.catch_contexts = []; - unpack_destructuring({ contexts: this.catch_contexts, node: info.error, scope, component, context_rest_properties: this.context_rest_properties, owner: this }); + unpack_destructuring({ contexts: this.catch_contexts, node: info.error, scope, component, context_rest_properties: this.context_rest_properties }); } this.pending = new PendingBlock(component, this, scope, info.pending); diff --git a/src/compiler/compile/nodes/ConstTag.ts b/src/compiler/compile/nodes/ConstTag.ts index 2163670db7..edcb949c15 100644 --- a/src/compiler/compile/nodes/ConstTag.ts +++ b/src/compiler/compile/nodes/ConstTag.ts @@ -61,8 +61,7 @@ export default class ConstTag extends Node { node: this.node.expression.left, scope: this.scope, component: this.component, - context_rest_properties: this.context_rest_properties, - owner: this + context_rest_properties: this.context_rest_properties }); this.expression = new Expression(this.component, this, this.scope, this.node.expression.right); this.contexts.forEach(context => { diff --git a/src/compiler/compile/nodes/EachBlock.ts b/src/compiler/compile/nodes/EachBlock.ts index 69584b16b8..4659a53d73 100644 --- a/src/compiler/compile/nodes/EachBlock.ts +++ b/src/compiler/compile/nodes/EachBlock.ts @@ -42,7 +42,7 @@ export default class EachBlock extends AbstractBlock { this.scope = scope.child(); this.context_rest_properties = new Map(); this.contexts = []; - unpack_destructuring({ contexts: this.contexts, node: info.context, scope, component, owner: this, context_rest_properties: this.context_rest_properties }); + unpack_destructuring({ contexts: this.contexts, node: info.context, scope, component, context_rest_properties: this.context_rest_properties }); this.contexts.forEach(context => { if (context.type !== 'DestructuredVariable') return; diff --git a/src/compiler/compile/nodes/shared/Context.ts b/src/compiler/compile/nodes/shared/Context.ts index 2194c03124..db92706122 100644 --- a/src/compiler/compile/nodes/shared/Context.ts +++ b/src/compiler/compile/nodes/shared/Context.ts @@ -5,7 +5,6 @@ import is_reference, { NodeWithPropertyDefinition } from 'is-reference'; import { clone } from '../../../utils/clone'; import Component from '../../Component'; import flatten_reference from '../../utils/flatten_reference'; -import { INode } from '../interfaces'; import TemplateScope from './TemplateScope'; export type Context = DestructuredVariable | ComputedProperty; @@ -32,7 +31,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props = 0 }: { contexts: Context[]; @@ -42,7 +40,6 @@ export function unpack_destructuring({ scope: TemplateScope; component: Component; context_rest_properties: Map; - owner: INode; number_of_computed_props?: number; }) { if (!node) return; @@ -73,7 +70,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props }); context_rest_properties.set((element.argument as Identifier).name, element); @@ -95,7 +91,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props }); } else { @@ -107,7 +102,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props }); } @@ -128,7 +122,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props }); context_rest_properties.set((property.argument as Identifier).name, property); @@ -183,7 +176,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props }); } else { @@ -196,7 +188,6 @@ export function unpack_destructuring({ scope, component, context_rest_properties, - owner, number_of_computed_props }); }