use ReadonlyArray for tags, microsoft#50972

jrieken · jrieken · commit f7fe25dcfc77 · 2019-08-23T08:46:25.000+02:00
diff --git a/src/vs/base/common/map.ts b/src/vs/base/common/map.ts
@@ -8,14 +8,6 @@ import { CharCode } from 'vs/base/common/charCode';
 import { Iterator, IteratorResult, FIN } from './iterator';
 
 
-export function fromArray<T>(array: readonly T[]): Set<T> {
-	const result = new Set<T>();
-	for (const element of array) {
-		result.add(element);
-	}
-	return result;
-}
-
 export function values<V = any>(set: Set<V>): V[];
 export function values<K = any, V = any>(map: Map<K, V>): V[];
 export function values<V>(forEachable: { forEach(callback: (value: V, ...more: any[]) => any): void }): V[] {
diff --git a/src/vs/editor/common/modes.ts b/src/vs/editor/common/modes.ts
@@ -404,7 +404,7 @@ export interface CompletionItem {
 	 * A modifier to the `kind` which affect how the item
 	 * is rendered, e.g. Deprecated is rendered with a strikeout
 	 */
-	kindTags?: Set<CompletionItemKindTag>;
+	kindTags?: ReadonlyArray<CompletionItemKindTag>;
 	/**
 	 * A human-readable string with additional information
 	 * about this item, like type or symbol information.
@@ -913,7 +913,7 @@ export interface DocumentSymbol {
 	name: string;
 	detail: string;
 	kind: SymbolKind;
-	kindTags: SymbolKindTag[];
+	kindTags: ReadonlyArray<SymbolKindTag>;
 	containerName?: string;
 	range: IRange;
 	selectionRange: IRange;
diff --git a/src/vs/editor/contrib/suggest/suggestWidget.ts b/src/vs/editor/contrib/suggest/suggestWidget.ts
@@ -193,7 +193,7 @@ class Renderer implements IListRenderer<CompletionItem, ISuggestionTemplateData>
 			];
 		}
 
-		if (suggestion.kindTags && suggestion.kindTags.has(CompletionItemKindTag.Deprecated)) {
+		if (suggestion.kindTags && suggestion.kindTags.indexOf(CompletionItemKindTag.Deprecated) >= 0) {
 			labelOptions.extraClasses = (labelOptions.extraClasses || []).concat(['deprecated']);
 			labelOptions.matches = [];
 		}
diff --git a/src/vs/monaco.d.ts b/src/vs/monaco.d.ts
@@ -4822,7 +4822,7 @@ declare namespace monaco.languages {
 		 * A modifier to the `kind` which affect how the item
 		 * is rendered, e.g. Deprecated is rendered with a strikeout
 		 */
-		kindTags?: Set<CompletionItemKindTag>;
+		kindTags?: ReadonlyArray<CompletionItemKindTag>;
 		/**
 		 * A human-readable string with additional information
 		 * about this item, like type or symbol information.
@@ -5240,7 +5240,7 @@ declare namespace monaco.languages {
 		name: string;
 		detail: string;
 		kind: SymbolKind;
-		kindTags: SymbolKindTag[];
+		kindTags: ReadonlyArray<SymbolKindTag>;
 		containerName?: string;
 		range: IRange;
 		selectionRange: IRange;
diff --git a/src/vs/workbench/api/browser/mainThreadLanguageFeatures.ts b/src/vs/workbench/api/browser/mainThreadLanguageFeatures.ts
@@ -21,7 +21,6 @@ import { Selection } from 'vs/editor/common/core/selection';
 import { ExtensionIdentifier } from 'vs/platform/extensions/common/extensions';
 import * as callh from 'vs/workbench/contrib/callHierarchy/common/callHierarchy';
 import { mixin } from 'vs/base/common/objects';
-import { fromArray } from 'vs/base/common/map';
 
 @extHostNamedCustomer(MainContext.MainThreadLanguageFeatures)
 export class MainThreadLanguageFeatures implements MainThreadLanguageFeaturesShape {
@@ -331,7 +330,7 @@ export class MainThreadLanguageFeatures implements MainThreadLanguageFeaturesSha
 		return {
 			label: data.a,
 			kind: data.b,
-			kindTags: data.n && fromArray(data.n),
+			kindTags: data.n,
 			detail: data.c,
 			documentation: data.d,
 			sortText: data.e,

Original file line number	Diff line number	Diff line change
`@@ -193,7 +193,7 @@ class Renderer implements IListRenderer<CompletionItem, ISuggestionTemplateData>`
`193`	`193`	`];`
`194`	`194`	`}`
`195`	`195`
`196`		`- if (suggestion.kindTags && suggestion.kindTags.has(CompletionItemKindTag.Deprecated)) {`
	`196`	`+ if (suggestion.kindTags && suggestion.kindTags.indexOf(CompletionItemKindTag.Deprecated) >= 0) {`
`197`	`197`	`labelOptions.extraClasses = (labelOptions.extraClasses \|\| []).concat(['deprecated']);`
`198`	`198`	`labelOptions.matches = [];`
`199`	`199`	`}`