From 144c096ae6dd4e3baf3520ebfff694bcce722da7 Mon Sep 17 00:00:00 2001
From: Dan Holmsand <holmsand@gmail.com>
Date: Fri, 30 Apr 2010 09:48:30 +0200
Subject: [PATCH] Remove unused variable index_var from for loops

---
 lib/cake.js      | 6 +++---
 lib/nodes.js     | 5 +----
 lib/optparse.js  | 6 +++---
 lib/rewriter.js  | 6 +++---
 src/nodes.coffee | 3 ---
 5 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/lib/cake.js b/lib/cake.js
index 3737698f..c04f7678 100755
--- a/lib/cake.js
+++ b/lib/cake.js
@@ -78,15 +78,15 @@
   };
   // Display the list of Cake tasks in a format similar to `rake -T`
   print_tasks = function print_tasks() {
-    var _a, _b, _c, _d, _e, desc, i, name, spaces, task;
+    var _a, _b, _c, _d, desc, i, name, spaces, task;
     puts('');
     _a = tasks;
     for (name in _a) { if (__hasProp.call(_a, name)) {
       task = _a[name];
       spaces = 20 - name.length;
       spaces = spaces > 0 ? (function() {
-        _b = []; _d = 0; _e = spaces;
-        for (_c = 0, i = _d; (_d <= _e ? i <= _e : i >= _e); (_d <= _e ? i += 1 : i -= 1), _c++) {
+        _b = []; _c = 0; _d = spaces;
+        for (i = _c; (_c <= _d ? i <= _d : i >= _d); (_c <= _d ? i += 1 : i -= 1)) {
           _b.push(' ');
         }
         return _b;
diff --git a/lib/nodes.js b/lib/nodes.js
index 411a9672..371541f8 100644
--- a/lib/nodes.js
+++ b/lib/nodes.js
@@ -1498,7 +1498,7 @@
     // comprehensions. Some of the generated code can be shared in common, and
     // some cannot.
     ForNode.prototype.compile_node = function compile_node(o) {
-      var body, body_dent, close, for_part, index, index_var, ivar, lvar, name, range, return_result, rvar, scope, set_result, source, source_part, step_part, svar, top_level, var_part, vars;
+      var body, body_dent, close, for_part, index, ivar, lvar, name, range, return_result, rvar, scope, set_result, source, source_part, step_part, svar, top_level, var_part, vars;
       top_level = del(o, 'top') && !this.returns;
       range = this.source instanceof ValueNode && this.source.base instanceof RangeNode && !this.source.properties.length;
       source = range ? this.source.base : this.source;
@@ -1519,16 +1519,13 @@
       var_part = '';
       body = Expressions.wrap([this.body]);
       if (range) {
-        index_var = scope.free_variable();
         source_part = source.compile_variables(o);
         for_part = source.compile(merge(o, {
           index: ivar,
           step: this.step
         }));
-        for_part = ("" + index_var + " = 0, " + for_part + ", " + index_var + "++");
       } else {
         svar = scope.free_variable();
-        index_var = null;
         source_part = ("" + svar + " = " + (this.source.compile(o)) + ";\n" + this.tab);
         if (this.pattern) {
           var_part = new AssignNode(this.name, literal(("" + svar + "[" + ivar + "]"))).compile(merge(o, {
diff --git a/lib/optparse.js b/lib/optparse.js
index db54d3e6..685eff0e 100755
--- a/lib/optparse.js
+++ b/lib/optparse.js
@@ -48,7 +48,7 @@
     // Return the help text for this **OptionParser**, listing and describing all
     // of the valid options, for `--help` and such.
     OptionParser.prototype.help = function help() {
-      var _a, _b, _c, _d, _e, _f, _g, i, let_part, lines, rule, spaces;
+      var _a, _b, _c, _d, _e, _f, i, let_part, lines, rule, spaces;
       lines = ['Available options:'];
       if (this.banner) {
         lines.unshift(("" + this.banner + "\n"));
@@ -58,8 +58,8 @@
         rule = _b[_a];
         spaces = 15 - rule.long_flag.length;
         spaces = spaces > 0 ? (function() {
-          _d = []; _f = 0; _g = spaces;
-          for (_e = 0, i = _f; (_f <= _g ? i <= _g : i >= _g); (_f <= _g ? i += 1 : i -= 1), _e++) {
+          _d = []; _e = 0; _f = spaces;
+          for (i = _e; (_e <= _f ? i <= _f : i >= _f); (_e <= _f ? i += 1 : i -= 1)) {
             _d.push(' ');
           }
           return _d;
diff --git a/lib/rewriter.js b/lib/rewriter.js
index a7799741..b28a1779 100644
--- a/lib/rewriter.js
+++ b/lib/rewriter.js
@@ -145,9 +145,9 @@
       var close_calls, stack;
       stack = [0];
       close_calls = __bind(function(i) {
-          var _b, _c, _d, size, tmp;
-          _c = 0; _d = stack[stack.length - 1];
-          for (_b = 0, tmp = _c; (_c <= _d ? tmp < _d : tmp > _d); (_c <= _d ? tmp += 1 : tmp -= 1), _b++) {
+          var _b, _c, size, tmp;
+          _b = 0; _c = stack[stack.length - 1];
+          for (tmp = _b; (_b <= _c ? tmp < _c : tmp > _c); (_b <= _c ? tmp += 1 : tmp -= 1)) {
             this.tokens.splice(i, 0, ['CALL_END', ')', this.tokens[i][2]]);
           }
           size = stack[stack.length - 1] + 1;
diff --git a/src/nodes.coffee b/src/nodes.coffee
index 9e96ae48..4c3cc544 100644
--- a/src/nodes.coffee
+++ b/src/nodes.coffee
@@ -1118,13 +1118,10 @@ exports.ForNode: class ForNode extends BaseNode
     var_part:       ''
     body:           Expressions.wrap([@body])
     if range
-      index_var:    scope.free_variable()
       source_part:  source.compile_variables o
       for_part:     source.compile merge o, {index: ivar, step: @step}
-      for_part:     "$index_var = 0, $for_part, $index_var++"
     else
       svar:         scope.free_variable()
-      index_var:    null
       source_part:  "$svar = ${ @source.compile(o) };\n$@tab"
       if @pattern
         var_part:   new AssignNode(@name, literal("$svar[$ivar]")).compile(merge o, {indent: @idt(1), top: true}) + "\n"