Merge pull request #4462 from erik-krogh/strayTodo

erik-krogh · web-flow · commit 83d6d6041ac9 · 2020-10-13T12:50:00.000+02:00
JS: remove stray todo
diff --git a/javascript/ql/src/Declarations/DeadStoreOfProperty.ql b/javascript/ql/src/Declarations/DeadStoreOfProperty.ql
@@ -79,7 +79,7 @@ predicate isAPropertyRead(Expr e, string name) {
  * Holds if `e` is a property write of a property `name`.
  */
 predicate isAPropertyWrite(ControlFlowNode e, string name) {
-  exists(DataFlow::PropWrite write | write.getWriteNode() = e and write.getPropertyName() = name) // TODO: umambi?
+  exists(DataFlow::PropWrite write | write.getWriteNode() = e and write.getPropertyName() = name)
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -79,7 +79,7 @@ predicate isAPropertyRead(Expr e, string name) {`
`79`	`79`	* Holds if `e` is a property write of a property `name`.
`80`	`80`	`*/`
`81`	`81`	`predicate isAPropertyWrite(ControlFlowNode e, string name) {`
`82`		`- exists(DataFlow::PropWrite write \| write.getWriteNode() = e and write.getPropertyName() = name) // TODO: umambi?`
	`82`	`+ exists(DataFlow::PropWrite write \| write.getWriteNode() = e and write.getPropertyName() = name)`
`83`	`83`	`}`
`84`	`84`
`85`	`85`	`/**`