refactor: rename i to startPartIndex

This commit is contained in:
Nils Knappmeier
2023-08-04 22:59:35 +02:00
committed by Jay Linski
parent 8c9f866655
commit e497a35d7f
+12 -6
View File
@@ -535,16 +535,22 @@ JavaScriptCompiler.prototype = {
this.resolvePath('data', parts, 0, true, strict); this.resolvePath('data', parts, 0, true, strict);
}, },
resolvePath: function(type, parts, i, falsy, strict) { resolvePath: function(type, parts, startPartIndex, falsy, strict) {
if (this.options.strict || this.options.assumeObjects) { if (this.options.strict || this.options.assumeObjects) {
this.push( this.push(
strictLookup(this.options.strict && strict, this, parts, i, type) strictLookup(
this.options.strict && strict,
this,
parts,
startPartIndex,
type
)
); );
return; return;
} }
let len = parts.length; let len = parts.length;
for (; i < len; i++) { for (let i = startPartIndex; i < len; i++) {
/* eslint-disable no-loop-func */ /* eslint-disable no-loop-func */
this.replaceStack(current => { this.replaceStack(current => {
let lookup = this.nameLookup(current, parts[i], type); let lookup = this.nameLookup(current, parts[i], type);
@@ -1263,14 +1269,14 @@ JavaScriptCompiler.isValidJavaScriptVariableName = function(name) {
); );
}; };
function strictLookup(requireTerminal, compiler, parts, i, type) { function strictLookup(requireTerminal, compiler, parts, startPartIndex, type) {
let stack = compiler.popStack(), let stack = compiler.popStack(),
len = parts.length; len = parts.length;
if (requireTerminal) { if (requireTerminal) {
len--; len--;
} }
for (; i < len; i++) { for (let i = startPartIndex; i < len; i++) {
stack = compiler.nameLookup(stack, parts[i], type); stack = compiler.nameLookup(stack, parts[i], type);
} }
@@ -1280,7 +1286,7 @@ function strictLookup(requireTerminal, compiler, parts, i, type) {
'(', '(',
stack, stack,
', ', ', ',
compiler.quotedString(parts[i]), compiler.quotedString(parts[len]),
', ', ', ',
JSON.stringify(compiler.source.currentLocation), JSON.stringify(compiler.source.currentLocation),
' )' ' )'