removed unsed variable and made DocumentLinter switch statement consistent

Author: lifeinData

plugin/vscode/extension.js

@@ -130,11 +130,13 @@ class DocumentLinter {
     switch (this._state) {
       case DocumentLinterState.NO_PARSER:
         await this._createParser();
-        return await this.editorChangedVisibilityAsync();
+        await this.editorChangedVisibilityAsync();
+        break;
 
       case DocumentLinterState.CREATING_PARSER:
         await this._parserPromise;
-        return await this.editorChangedVisibilityAsync();
+        await this.editorChangedVisibilityAsync();
+        break;
 
       case DocumentLinterState.PARSER_UNINITIALIZED:
         await this._initializeParserAsync();
@@ -347,7 +349,6 @@ class DocumentLinterCollection {
   }
 
   async disposeAsync() {
-    let linters = this._linters;
     this._linters = new Map();
     for (let [_uri, linter] of this._linters) {
       await linter.disposeAsync();