move disposeDecorators call into setDecorators (#2068)

microsoft · Nov 1, 2018 · c96594b · c96594b
1 parent 3ba34e0
commit c96594b
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/goCover.ts b/src/goCover.ts
@@ -89,11 +89,11 @@ export function updateCodeCoverageDecorators(coverageDecoratorConfig: any) {
 			decoratorConfig[k] = coverageDecoratorConfig[k];
 		}
 	}
-	disposeDecorators();
 	setDecorators();
 }
 
 function setDecorators() {
+	disposeDecorators();
 	decorators = {
 		type: decoratorConfig.type,
 		coveredGutterDecorator: vscode.window.createTextEditorDecorationType({ gutterIconPath: gutterSvgs[decoratorConfig.coveredGutterStyle] }),
@@ -234,7 +234,6 @@ export function applyCodeCoverage(editor: vscode.TextEditor) {
 
 	const cfg = vscode.workspace.getConfiguration('go', editor.document.uri);
 	const coverageOptions = cfg['coverageOptions'];
-	disposeDecorators();
 	setDecorators();
 
 	for (let filename in coverageFiles) {