Remove TODO.

starcke · starcke · commit bb258742992a · 2023-09-28T14:05:31.000+02:00
diff --git a/extensions/ql-vscode/src/language-context-store.ts b/extensions/ql-vscode/src/language-context-store.ts
@@ -33,7 +33,10 @@ export class LanguageContextStore extends DisposableObject {
     this.onLanguageContextChangedEmitter.fire();
   }
 
-  // TODO: comment on why string is used here
+  // This method takes a string to allow it to be used in cases
+  // where the language is not always a known one.
+  // The semantics of such an unknown langauge is that it is
+  // only included if the current language context is "All".
   public shouldInclude(language: string): boolean {
     return this.state === "All" || this.state === language;
   }

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,10 @@ export class LanguageContextStore extends DisposableObject {`
`33`	`33`	`this.onLanguageContextChangedEmitter.fire();`
`34`	`34`	`}`
`35`	`35`
`36`		`- // TODO: comment on why string is used here`
	`36`	`+ // This method takes a string to allow it to be used in cases`
	`37`	`+ // where the language is not always a known one.`
	`38`	`+ // The semantics of such an unknown langauge is that it is`
	`39`	`+ // only included if the current language context is "All".`
`37`	`40`	`public shouldInclude(language: string): boolean {`
`38`	`41`	`return this.state === "All" \|\| this.state === language;`
`39`	`42`	`}`