Merge pull request #42914 from Connormiha/optimize-context-key-change…

…-event Remove needless check for array in ContextKeyChangeEvent
microsoft · Feb 26, 2018 · 43fbb43 · 43fbb43
2 parents dbbfa5d + a75736e
commit 43fbb43
Showing 1 changed file with 1 addition and 5 deletions.
diff --git a/src/vs/platform/contextkey/browser/contextKeyService.ts b/src/vs/platform/contextkey/browser/contextKeyService.ts
@@ -176,11 +176,7 @@ export class ContextKeyChangeEvent implements IContextKeyChangeEvent {
 	private _keys: string[] = [];
 
 	collect(oneOrManyKeys: string | string[]): void {
-		if (Array.isArray(oneOrManyKeys)) {
-			this._keys = this._keys.concat(oneOrManyKeys);
-		} else {
-			this._keys.push(oneOrManyKeys);
-		}
+		this._keys = this._keys.concat(oneOrManyKeys);
 	}
 
 	affectsSome(keys: Set<string>): boolean {