cargdev/angular
1
0
Fork 0
Code Issues 16 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(compiler): add PURE annotation to getInheritedFactory calls (#38291)

Browse Source 
Currently the `getInheritedFactory` function is implemented to allow
closure to remove the call if the base factory is unused.  However, this
method does not work with terser.  By adding the PURE annotation,
terser will also be able to remove the call when unused.

PR Close #38291
This commit is contained in:
Charles Lyding
2020-07-29 15:31:27 -04:00
committed by Michael Prentice
parent e1ee231993
commit e169f00193
4 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
packages/compiler/src/output/output_ast.ts
View File

@ -176,8 +176,9 @@ export abstract class Expression {
return new InvokeMethodExpr(this, name, params, null, sourceSpan);
}
callFn(params: Expression[], sourceSpan?: ParseSourceSpan|null): InvokeFunctionExpr {
return new InvokeFunctionExpr(this, params, null, sourceSpan);
callFn(params: Expression[], sourceSpan?: ParseSourceSpan|null, pure?: boolean):
InvokeFunctionExpr {
return new InvokeFunctionExpr(this, params, null, sourceSpan, pure);
}
instantiate(params: Expression[], type?: Type|null, sourceSpan?: ParseSourceSpan|null):

4
packages/compiler/src/render3/r3_factory.ts
View File

@ -212,7 +212,9 @@ export function compileFactoryFunction(meta: R3FactoryMetadata): R3FactoryFn {
const baseFactory = o.variable(`ɵ${meta.name}_BaseFactory`);
const getInheritedFactory = o.importExpr(R3.getInheritedFactory);
const baseFactoryStmt =
baseFactory.set(getInheritedFactory.callFn([meta.internalType]))
baseFactory
.set(getInheritedFactory.callFn(
[meta.internalType], /* sourceSpan */ undefined, /* pure */ true))
.toDeclStmt(o.INFERRED_TYPE, [o.StmtModifier.Exported, o.StmtModifier.Final]);
statements.push(baseFactoryStmt);