Fix variable scoping

semenko · semenko · commit 558cd6f373d6 · 2016-01-25T13:46:21.000-06:00
Signed-off-by: Nick Semenkovich &lt;semenko@alum.mit.edu&gt;
diff --git a/chromium/rules.js b/chromium/rules.js
@@ -261,9 +261,8 @@ RuleSets.prototype = {
     }
     // now eat away from the left, with *, so that for x.y.z.google.com we
     // check *.z.google.com and *.google.com (we did *.y.z.google.com above)
-    var t;
     for (var i = 2; i <= segmented.length - 2; ++i) {
-      t = "*." + segmented.slice(i,segmented.length).join(".");
+      var t = "*." + segmented.slice(i,segmented.length).join(".");
       this.setInsert(results, this.targets[t]);
     }
     log(DBUG,"Applicable rules for " + host + ":");

Original file line number	Diff line number	Diff line change
`@@ -261,9 +261,8 @@ RuleSets.prototype = {`
`261`	`261`	`}`
`262`	`262`	`// now eat away from the left, with *, so that for x.y.z.google.com we`
`263`	`263`	`// check .z.google.com and .google.com (we did *.y.z.google.com above)`
`264`		`- var t;`
`265`	`264`	`for (var i = 2; i <= segmented.length - 2; ++i) {`
`266`		`- t = "*." + segmented.slice(i,segmented.length).join(".");`
	`265`	`+ var t = "*." + segmented.slice(i,segmented.length).join(".");`
`267`	`266`	`this.setInsert(results, this.targets[t]);`
`268`	`267`	`}`
`269`	`268`	`log(DBUG,"Applicable rules for " + host + ":");`