mrva/vscode-codeql
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Remove unnecessary if-statements before slice

Browse Source
This commit is contained in:
Koen Vlaswinkel
2023-05-25 15:52:00 +02:00
parent a88e683ebf
commit adcc3d0b39
1 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
extensions/ql-vscode/src/data-extensions-editor/auto-model.ts
View File

@@ -48,14 +48,12 @@ export function createAutoModelRequest(
modeledMethod.type === "none" modeledMethod.type === "none"
? undefined ? undefined
: toMethodClassification(modeledMethod), : toMethodClassification(modeledMethod),
usages: externalApiUsage.usages.map((usage) => usage.label), usages: externalApiUsage.usages
.slice(0, 10)
.map((usage) => usage.label),
input: `Argument[${argumentIndex}]`, input: `Argument[${argumentIndex}]`,
}; };
if (method.usages.length > 10) {
method.usages = method.usages.slice(0, 10);
}
if (modeledMethod.type === "none") { if (modeledMethod.type === "none") {
request.candidates.push(method); request.candidates.push(method);
} else { } else {
@@ -64,12 +62,8 @@ export function createAutoModelRequest(
} }
} }
if (request.candidates.length > 100) { request.candidates = request.candidates.slice(0, 100);
request.candidates = request.candidates.slice(0, 100); request.samples = request.samples.slice(0, 20);
}
if (request.samples.length > 20) {
request.samples = request.samples.slice(0, 20);
}
return request; return request;
} }