JS: Remove obsolete TODO

asgerf · asgerf · commit 7766f97232a7 · 2025-01-09T09:39:26.000+01:00
diff --git a/javascript/ql/lib/semmle/javascript/dataflow/internal/VariableCapture.qll b/javascript/ql/lib/semmle/javascript/dataflow/internal/VariableCapture.qll
@@ -114,7 +114,6 @@ module VariableCaptureConfig implements InputSig<js::DbLocation> {
 
   class Callable extends js::StmtContainer {
     predicate isConstructor() {
-      // TODO: clarify exactly what the library wants to know here as the meaning of "constructor" varies between languages.
       // JS constructors should not be seen as "constructors" in this context.
       none()
     }

Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,6 @@ module VariableCaptureConfig implements InputSig<js::DbLocation> {`
`114`	`114`
`115`	`115`	`class Callable extends js::StmtContainer {`
`116`	`116`	`predicate isConstructor() {`
`117`		`- // TODO: clarify exactly what the library wants to know here as the meaning of "constructor" varies between languages.`
`118`	`117`	`// JS constructors should not be seen as "constructors" in this context.`
`119`	`118`	`none()`
`120`	`119`	`}`