From ab3f2e6180dbfce96bf16917103a1c07e4abbae2 Mon Sep 17 00:00:00 2001 From: Matt Bierner Date: Mon, 7 Oct 2019 11:10:23 -0700 Subject: [PATCH] Use find instead of for loop --- src/vs/workbench/browser/composite.ts | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/vs/workbench/browser/composite.ts b/src/vs/workbench/browser/composite.ts index fdbd1478f1b..6414496a6d3 100644 --- a/src/vs/workbench/browser/composite.ts +++ b/src/vs/workbench/browser/composite.ts @@ -15,6 +15,7 @@ import { trackFocus, Dimension } from 'vs/base/browser/dom'; import { IStorageService } from 'vs/platform/storage/common/storage'; import { Disposable } from 'vs/base/common/lifecycle'; import { assertIsDefined } from 'vs/base/common/types'; +import { find } from 'vs/base/common/arrays'; /** * Composites are layed out in the sidebar and panel part of the workbench. At a time only one composite @@ -285,12 +286,6 @@ export abstract class CompositeRegistry extends Disposable } private compositeById(id: string): CompositeDescriptor | undefined { - for (const composite of this.composites) { - if (composite.id === id) { - return composite; - } - } - - return undefined; + return find(this.composites, composite => composite.id === id); } }