JS: Mention why we cache predicates outside the class

asgerf · asgerf · commit c2764069b567 · 2021-03-17T13:29:17.000Z
diff --git a/javascript/ql/src/semmle/javascript/dataflow/TaintTracking.qll b/javascript/ql/src/semmle/javascript/dataflow/TaintTracking.qll
@@ -308,7 +308,8 @@ module TaintTracking {
   }
 
   /**
-   * Module existing only to ensure all taint steps are cached as a single stage.
+   * Module existing only to ensure all taint steps are cached as a single stage,
+   * and without the the `Unit` type column.
    */
   cached
   private module Cached {

Original file line number	Diff line number	Diff line change
`@@ -308,7 +308,8 @@ module TaintTracking {`
`308`	`308`	`}`
`309`	`309`
`310`	`310`	`/**`
`311`		`- * Module existing only to ensure all taint steps are cached as a single stage.`
	`311`	`+ * Module existing only to ensure all taint steps are cached as a single stage,`
	`312`	+ * and without the the `Unit` type column.
`312`	`313`	`*/`
`313`	`314`	`cached`
`314`	`315`	`private module Cached {`