Avoid firing unnecessary SearchResult changed events

roblourens · roblourens · commit 32942fd950cf · 2017-03-21T19:19:11.000-07:00
diff --git a/src/vs/workbench/parts/search/common/searchModel.ts b/src/vs/workbench/parts/search/common/searchModel.ts
@@ -362,7 +362,7 @@ export class SearchResult extends Disposable {
 				fileMatch.onDispose(() => disposable.dispose());
 			}
 		});
-		if (!silent) {
+		if (!silent && changed.length) {
 			this._onChange.fire({ elements: changed, added: true });
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -362,7 +362,7 @@ export class SearchResult extends Disposable {`
`362`	`362`	`fileMatch.onDispose(() => disposable.dispose());`
`363`	`363`	`}`
`364`	`364`	`});`
`365`		`- if (!silent) {`
	`365`	`+ if (!silent && changed.length) {`
`366`	`366`	`this._onChange.fire({ elements: changed, added: true });`
`367`	`367`	`}`
`368`	`368`	`}`