From 376ec0634152d9c7e7944f29d1588b2f22a49c6a Mon Sep 17 00:00:00 2001 From: Matt Bierner Date: Fri, 25 Sep 2020 16:27:58 -0700 Subject: [PATCH] Replace strings.repeat Use standard ''.repeat instead --- src/vs/base/common/strings.ts | 11 ----------- src/vs/base/test/common/strings.test.ts | 7 ------- .../workbench/contrib/search/browser/searchActions.ts | 5 ++--- .../searchEditor/browser/searchEditorSerialization.ts | 5 ++--- 4 files changed, 4 insertions(+), 24 deletions(-) diff --git a/src/vs/base/common/strings.ts b/src/vs/base/common/strings.ts index fce6855c8b0..297873577dc 100644 --- a/src/vs/base/common/strings.ts +++ b/src/vs/base/common/strings.ts @@ -820,17 +820,6 @@ export function stripUTF8BOM(str: string): string { return startsWithUTF8BOM(str) ? str.substr(1) : str; } -/** - * @deprecated ES6 - */ -export function repeat(s: string, count: number): string { - let result = ''; - for (let i = 0; i < count; i++) { - result += s; - } - return result; -} - /** * Checks if the characters of the provided query string are included in the * target string. The characters do not have to be contiguous within the string. diff --git a/src/vs/base/test/common/strings.test.ts b/src/vs/base/test/common/strings.test.ts index 9a6d9d75308..35ba912d0cb 100644 --- a/src/vs/base/test/common/strings.test.ts +++ b/src/vs/base/test/common/strings.test.ts @@ -183,13 +183,6 @@ suite('Strings', () => { assert.strictEqual(' '.trim(), ''); }); - test('repeat', () => { - assert.strictEqual(strings.repeat(' ', 4), ' '); - assert.strictEqual(strings.repeat(' ', 1), ' '); - assert.strictEqual(strings.repeat(' ', 0), ''); - assert.strictEqual(strings.repeat('abc', 2), 'abcabc'); - }); - test('lastNonWhitespaceIndex', () => { assert.strictEqual(strings.lastNonWhitespaceIndex('abc \t \t '), 2); assert.strictEqual(strings.lastNonWhitespaceIndex('abc'), 2); diff --git a/src/vs/workbench/contrib/search/browser/searchActions.ts b/src/vs/workbench/contrib/search/browser/searchActions.ts index 040baac60b9..0c3470b966b 100644 --- a/src/vs/workbench/contrib/search/browser/searchActions.ts +++ b/src/vs/workbench/contrib/search/browser/searchActions.ts @@ -7,7 +7,6 @@ import * as DOM from 'vs/base/browser/dom'; import { Action } from 'vs/base/common/actions'; import { createKeybinding, ResolvedKeybinding } from 'vs/base/common/keyCodes'; import { isWindows, OS } from 'vs/base/common/platform'; -import { repeat } from 'vs/base/common/strings'; import * as nls from 'vs/nls'; import { IClipboardService } from 'vs/platform/clipboard/common/clipboardService'; import { ILabelService } from 'vs/platform/label/common/label'; @@ -804,8 +803,8 @@ function matchToString(match: Match, indent = 0): string { getFirstLinePrefix() : getOtherLinePrefix(i); - const paddingStr = repeat(' ', largestPrefixSize - prefix.length); - const indentStr = repeat(' ', indent); + const paddingStr = ' '.repeat(largestPrefixSize - prefix.length); + const indentStr = ' '.repeat(indent); return `${indentStr}${prefix}: ${paddingStr}${line}`; }); diff --git a/src/vs/workbench/contrib/searchEditor/browser/searchEditorSerialization.ts b/src/vs/workbench/contrib/searchEditor/browser/searchEditorSerialization.ts index 31d48055864..91cf43f0323 100644 --- a/src/vs/workbench/contrib/searchEditor/browser/searchEditorSerialization.ts +++ b/src/vs/workbench/contrib/searchEditor/browser/searchEditorSerialization.ts @@ -4,7 +4,6 @@ *--------------------------------------------------------------------------------------------*/ import { coalesce, flatten } from 'vs/base/common/arrays'; -import { repeat } from 'vs/base/common/strings'; import { URI } from 'vs/base/common/uri'; import 'vs/css!./media/searchEditor'; import { ServicesAccessor } from 'vs/editor/browser/editorExtensions'; @@ -35,7 +34,7 @@ const matchToSearchResultFormat = (match: Match, longestLineNumber: number): { l fullMatchLines .forEach((sourceLine, i) => { const lineNumber = getLinePrefix(i); - const paddingStr = repeat(' ', longestLineNumber - lineNumber.length); + const paddingStr = ' '.repeat(longestLineNumber - lineNumber.length); const prefix = ` ${paddingStr}${lineNumber}: `; const prefixOffset = prefix.length; @@ -85,7 +84,7 @@ function fileMatchToSearchResultFormat(fileMatch: FileMatch, labelFormatter: (x: if (lastLine !== undefined && lineNumber !== lastLine + 1) { text.push(''); } - text.push(` ${repeat(' ', longestLineNumber - `${lineNumber}`.length)}${lineNumber} ${line}`); + text.push(` ${' '.repeat(longestLineNumber - `${lineNumber}`.length)}${lineNumber} ${line}`); lastLine = lineNumber; } -- GitLab