refactor: private methods should not be decorated with internal
Closes #5146
This commit is contained in:
parent
7e92d2e6b7
commit
b84855f394
|
@ -299,7 +299,6 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _calculateCurrValue(proto: ProtoRecord, values: any[], locals: Locals) {
|
private _calculateCurrValue(proto: ProtoRecord, values: any[], locals: Locals) {
|
||||||
switch (proto.mode) {
|
switch (proto.mode) {
|
||||||
case RecordType.Self:
|
case RecordType.Self:
|
||||||
|
@ -367,7 +366,6 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _pipeCheck(proto: ProtoRecord, throwOnChange: boolean, values: any[]) {
|
private _pipeCheck(proto: ProtoRecord, throwOnChange: boolean, values: any[]) {
|
||||||
var context = this._readContext(proto, values);
|
var context = this._readContext(proto, values);
|
||||||
var selectedPipe = this._pipeFor(proto, context);
|
var selectedPipe = this._pipeFor(proto, context);
|
||||||
|
@ -406,7 +404,6 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _pipeFor(proto: ProtoRecord, context) {
|
private _pipeFor(proto: ProtoRecord, context) {
|
||||||
var storedPipe = this._readPipe(proto);
|
var storedPipe = this._readPipe(proto);
|
||||||
if (isPresent(storedPipe)) return storedPipe;
|
if (isPresent(storedPipe)) return storedPipe;
|
||||||
|
@ -416,7 +413,6 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
|
||||||
return pipe;
|
return pipe;
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _readContext(proto: ProtoRecord, values: any[]) {
|
private _readContext(proto: ProtoRecord, values: any[]) {
|
||||||
if (proto.contextIndex == -1) {
|
if (proto.contextIndex == -1) {
|
||||||
return this._getDirectiveFor(proto.directiveIndex);
|
return this._getDirectiveFor(proto.directiveIndex);
|
||||||
|
@ -424,29 +420,22 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
|
||||||
return values[proto.contextIndex];
|
return values[proto.contextIndex];
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _readSelf(proto: ProtoRecord, values: any[]) { return values[proto.selfIndex]; }
|
private _readSelf(proto: ProtoRecord, values: any[]) { return values[proto.selfIndex]; }
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _writeSelf(proto: ProtoRecord, value, values: any[]) { values[proto.selfIndex] = value; }
|
private _writeSelf(proto: ProtoRecord, value, values: any[]) { values[proto.selfIndex] = value; }
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _readPipe(proto: ProtoRecord) { return this.localPipes[proto.selfIndex]; }
|
private _readPipe(proto: ProtoRecord) { return this.localPipes[proto.selfIndex]; }
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _writePipe(proto: ProtoRecord, value) { this.localPipes[proto.selfIndex] = value; }
|
private _writePipe(proto: ProtoRecord, value) { this.localPipes[proto.selfIndex] = value; }
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _setChanged(proto: ProtoRecord, value: boolean) {
|
private _setChanged(proto: ProtoRecord, value: boolean) {
|
||||||
if (proto.argumentToPureFunction) this.changes[proto.selfIndex] = value;
|
if (proto.argumentToPureFunction) this.changes[proto.selfIndex] = value;
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _pureFuncAndArgsDidNotChange(proto: ProtoRecord): boolean {
|
private _pureFuncAndArgsDidNotChange(proto: ProtoRecord): boolean {
|
||||||
return proto.isPureFunction() && !this._argsChanged(proto);
|
return proto.isPureFunction() && !this._argsChanged(proto);
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _argsChanged(proto: ProtoRecord): boolean {
|
private _argsChanged(proto: ProtoRecord): boolean {
|
||||||
var args = proto.args;
|
var args = proto.args;
|
||||||
for (var i = 0; i < args.length; ++i) {
|
for (var i = 0; i < args.length; ++i) {
|
||||||
|
@ -457,12 +446,10 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _argsOrContextChanged(proto: ProtoRecord): boolean {
|
private _argsOrContextChanged(proto: ProtoRecord): boolean {
|
||||||
return this._argsChanged(proto) || this.changes[proto.contextIndex];
|
return this._argsChanged(proto) || this.changes[proto.contextIndex];
|
||||||
}
|
}
|
||||||
|
|
||||||
/** @internal */
|
|
||||||
private _readArgs(proto: ProtoRecord, values: any[]) {
|
private _readArgs(proto: ProtoRecord, values: any[]) {
|
||||||
var res = ListWrapper.createFixedSize(proto.args.length);
|
var res = ListWrapper.createFixedSize(proto.args.length);
|
||||||
var args = proto.args;
|
var args = proto.args;
|
||||||
|
|
Loading…
Reference in New Issue