msb-public
/
svelte
mirror of https://github.com/sveltejs/svelte
1
0
Code Issues Projects Releases Wiki Activity

deduplicate each block context generation (#1287)

Browse Source
pull/1384/head
Rich Harris 6 years ago
parent dbab1a886d
commit 61ee380e26
3 changed files with 44 additions and 40 deletions
Show Stats Download Patch File Download Diff File

36
src/compile/nodes/EachBlock.ts
Unescape View File

@ -63,7 +63,7 @@ export default class EachBlock extends Node {
this.var = block.getUniqueName(`each`);
this.iterations = block.getUniqueName(`${this.var}_blocks`);
this.each_context = block.getUniqueName(`${this.var}_context`);
this.get_each_context = block.getUniqueName(`get_${this.var}_context`);
const { dependencies } = this.expression;
block.addDependencies(dependencies);
@ -88,14 +88,14 @@ export default class EachBlock extends Node {
}
this.contextProps = [
`${listName}: ${listName}`,
`${this.context}: ${listName}[#i]`,
`${indexName}: #i`
`${listName}: list`,
`${this.context}: list[i]`,
`${indexName}: i`
];
if (this.destructuredContexts) {
for (let i = 0; i < this.destructuredContexts.length; i += 1) {
this.contextProps.push(`${this.destructuredContexts[i]}: ${listName}[#i][${i}]`);
this.contextProps.push(`${this.destructuredContexts[i]}: list[i][${i}]`);
}
}
@ -162,6 +162,14 @@ export default class EachBlock extends Node {
block.builders.init.addLine(`var ${each_block_value} = ${snippet};`);
this.compiler.target.blocks.push(deindent`
function ${this.get_each_context}(ctx, list, i) {
return @assign(@assign({}, ctx), {
${this.contextProps.join(',\n')}
});
}
`);
if (this.key) {
this.buildKeyed(block, parentNode, parentNodes, snippet, vars);
} else {
@ -349,9 +357,7 @@ export default class EachBlock extends Node {
var ${iterations} = [];
for (var #i = 0; #i < ${each_block_value}.${length}; #i += 1) {
${iterations}[#i] = ${create_each_block}(#component, @assign(@assign({}, ctx), {
${this.contextProps.join(',\n')}
}));
${iterations}[#i] = ${create_each_block}(#component, ${this.get_each_context}(ctx, ${each_block_value}, #i));
}
`);
@ -395,24 +401,24 @@ export default class EachBlock extends Node {
? this.block.hasIntroMethod
? deindent`
if (${iterations}[#i]) {
${iterations}[#i].p(changed, ${this.each_context});
${iterations}[#i].p(changed, child_ctx);
} else {
${iterations}[#i] = ${create_each_block}(#component, ${this.each_context});
${iterations}[#i] = ${create_each_block}(#component, child_ctx);
${iterations}[#i].c();
}
${iterations}[#i].i(${updateMountNode}, ${anchor});
`
: deindent`
if (${iterations}[#i]) {
${iterations}[#i].p(changed, ${this.each_context});
${iterations}[#i].p(changed, child_ctx);
} else {
${iterations}[#i] = ${create_each_block}(#component, ${this.each_context});
${iterations}[#i] = ${create_each_block}(#component, child_ctx);
${iterations}[#i].c();
${iterations}[#i].m(${updateMountNode}, ${anchor});
}
`
: deindent`
${iterations}[#i] = ${create_each_block}(#component, ${this.each_context});
${iterations}[#i] = ${create_each_block}(#component, child_ctx);
${iterations}[#i].c();
${iterations}[#i].${mountOrIntro}(${updateMountNode}, ${anchor});
`;
@ -447,9 +453,7 @@ export default class EachBlock extends Node {
${each_block_value} = ${snippet};
for (var #i = ${start}; #i < ${each_block_value}.${length}; #i += 1) {
var ${this.each_context} = @assign(@assign({}, ctx), {
${this.contextProps.join(',\n')}
});
const child_ctx = ${this.get_each_context}(ctx, ${each_block_value}, #i);
${forLoopBody}
}

24
test/js/samples/each-block-changed-check/expected-bundle.js
Unescape View File

@ -163,11 +163,7 @@ function create_main_fragment(component, ctx) {
var each_blocks = [];
for (var i = 0; i < each_value.length; i += 1) {
each_blocks[i] = create_each_block(component, assign(assign({}, ctx), {
each_value: each_value,
comment: each_value[i],
i: i
}));
each_blocks[i] = create_each_block(component, get_each_context(ctx, each_value, i));
}
return {
@ -196,16 +192,12 @@ function create_main_fragment(component, ctx) {
each_value = ctx.comments;
for (var i = 0; i < each_value.length; i += 1) {
var each_context = assign(assign({}, ctx), {
each_value: each_value,
comment: each_value[i],
i: i
});
const child_ctx = get_each_context(ctx, each_value, i);
if (each_blocks[i]) {
each_blocks[i].p(changed, each_context);
each_blocks[i].p(changed, child_ctx);
} else {
each_blocks[i] = create_each_block(component, each_context);
each_blocks[i] = create_each_block(component, child_ctx);
each_blocks[i].c();
each_blocks[i].m(text.parentNode, text);
}
@ -303,6 +295,14 @@ function create_each_block(component, ctx) {
};
}
function get_each_context(ctx, list, i) {
return assign(assign({}, ctx), {
each_value: list,
comment: list[i],
i: i
});
}
function SvelteComponent(options) {
init(this, options);
this._state = assign({}, options.data);

24
test/js/samples/each-block-changed-check/expected.js
Unescape View File

@ -9,11 +9,7 @@ function create_main_fragment(component, ctx) {
var each_blocks = [];
for (var i = 0; i < each_value.length; i += 1) {
each_blocks[i] = create_each_block(component, assign(assign({}, ctx), {
each_value: each_value,
comment: each_value[i],
i: i
}));
each_blocks[i] = create_each_block(component, get_each_context(ctx, each_value, i));
}
return {
@ -42,16 +38,12 @@ function create_main_fragment(component, ctx) {
each_value = ctx.comments;
for (var i = 0; i < each_value.length; i += 1) {
var each_context = assign(assign({}, ctx), {
each_value: each_value,
comment: each_value[i],
i: i
});
const child_ctx = get_each_context(ctx, each_value, i);
if (each_blocks[i]) {
each_blocks[i].p(changed, each_context);
each_blocks[i].p(changed, child_ctx);
} else {
each_blocks[i] = create_each_block(component, each_context);
each_blocks[i] = create_each_block(component, child_ctx);
each_blocks[i].c();
each_blocks[i].m(text.parentNode, text);
}
@ -149,6 +141,14 @@ function create_each_block(component, ctx) {
};
}
function get_each_context(ctx, list, i) {
return assign(assign({}, ctx), {
each_value: list,
comment: list[i],
i: i
});
}
function SvelteComponent(options) {
init(this, options);
this._state = assign({}, options.data);

Write Preview
Loading…
Cancel
Save